You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First of all, thanks for your amazing work on this crate. It made my work a hell lot easier.
Currently, #[optfield(attrs)] only works on attributes declared after it. The ones before it are not picked up. This behaviour is confusing and is not mentioned anywhere in the documentation (I think).
Moreover, it is extremely rare for attribute order to be important in the entire Rust ecosystem, so I guess it's fair to say that this behaviour is not very idiomatic. I think it would be ideal if this is not the case, but if there are technical limitations that prevent a fix, this idiosyncrasy should at be noted in the documentation.
The text was updated successfully, but these errors were encountered:
Thanks for pointing this out!
I'm happy to hear that you found optfield useful.
I haven't touched it in quite some time. I now ran into some difficulties while updating the github actions, but will look into this issue after those are solved.
First of all, thanks for your amazing work on this crate. It made my work a hell lot easier.
Currently,
#[optfield(attrs)]
only works on attributes declared after it. The ones before it are not picked up. This behaviour is confusing and is not mentioned anywhere in the documentation (I think).Example:
Moreover, it is extremely rare for attribute order to be important in the entire Rust ecosystem, so I guess it's fair to say that this behaviour is not very idiomatic. I think it would be ideal if this is not the case, but if there are technical limitations that prevent a fix, this idiosyncrasy should at be noted in the documentation.
The text was updated successfully, but these errors were encountered: