The Wayback Machine - https://web.archive.org/web/20220517151219/https://github.com/RustPython/RustPython/issues/3637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[RFC] Rename derive macro to `py::` prefixed #3637

Open

youknowone opened this issue Apr 18, 2022 · 3 comments

Labels

RFC

Member

youknowone commented Apr 18, 2022 •

edited

Summary

#[pyclass] => #[py::class]
#[pyimpl] => #[py::class] // I think we can distinguish this from macro
#[pyattr] => #[py::attr]
#[pyfunction] => #[py::function]
#[pyslot] => #[py::slot]
#[pymethod] => #[py::method]
#[pyproperty] => #[py::property]

Detailed Explanation

Drawbacks, Rationale, and Alternatives

Unresolved Questions

The text was updated successfully, but these errors were encountered:

youknowone added the RFC label

Member

coolreader18 commented Apr 19, 2022

How would rustfmt deal with the colon?

youknowone changed the title ~~[RFC] Rename derive macro to py: prefixed~~ [RFC] Rename derive macro to py:: prefixed

Member Author

youknowone commented Apr 19, 2022

I added another colon to negotiate with rustfmt

youknowone mentioned this issue

poc for py:: prefix #3645

Draft

Member Author

youknowone commented Apr 29, 2022

@coolreader18 I hope we can do this before next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment