What I don't like here is that I have to repeat ident and literal names. Why?

• I cannot use just ident names with #[pyclass(name = ..., base = ...)], because they explicitly need literal strings. I even cannot use stringify!, because it is not expanded in #[pyclass] attributes
• I cannot use just literal, because Rust cannot unstrigify! things, unless https://docs.rs/unstringify/0.1.1/unstringify/macro.unstringify.html is installed

Friendly ping @youknowone @DimitrisJim

I don't any a nice idea for this. This problem is known to be a tranditional rust macro problem. So it looks just ok to me for now. Will it be better if pyclass takes both ident and str for its macro arguments? @coolreader18

I think so, #[pyclass(name = :ident | :literal, base = :ident | :literal)] seems like a good solution 👍
I will implement it in a separate PR.

Looks like that there's nothing I can do. It is a limitation of pub type AttributeArgs = Vec<NestedMeta>;, where NestedMeta can be:

    /// ## Path
    ///
    /// A meta path is like the `test` in `#[test]`.
    ///
    /// ## List
    ///
    /// A meta list is like the `derive(Copy)` in `#[derive(Copy)]`.
    ///
    /// ## NameValue
    ///
    /// A name-value meta is like the `path = "..."` in `#[path =
    /// "sys/windows.rs"]`.

And NameValue only allows Lit values 😞
So, I guess that the only way is to pass ident and literal as two separate params.

https://github.com/dtolnay/syn/blob/5a9efb0b772b583af93cdca9453fa9fd4e522641/src/lit.rs#L745

Hm #[pyexception($class_name, $base_class)] seems to work, I will continue my experiments 👍

Btw, here's another example why I need #2876
As you can see it uses 2 spaces and cargo fmt ignores it.

Test that it is not counted in module, I can leave it or remove it.