Skip to content

Rename detail namespace to auimpl #427

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

Closed
wants to merge 1 commit into from
Closed

Conversation

chiphogg
Copy link
Contributor

@chiphogg chiphogg commented May 9, 2025

I think a "bite the bullet megadiff" makes the most sense here, because
the changes should all be the same length, so the PR should be fairly
easy to review. The reviewer can check the Viewed box for one file at a
time, and thus not be forced to do it all in one pass --- as long as we
do end up landing this relatively soon before we try other changes.

Fixes #412.

I think a "bite the bullet megadiff" makes the most sense here, because
the changes should all be the same length, so the PR should be fairly
easy to review.  The reviewer can check the Viewed box for one file at a
time, and thus not be forced to do it all in one pass --- as long as we
do end up landing this relatively soon before we try other changes.
detail::OrderByOrigin,
detail::OrderAsUnitProduct,
detail::OrderAsOriginDisplacementUnit> {};
auimpl::OrderByUnitAvoidance,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'm going to skip doing this for now, because au::detail::UnitAvoidance is actually in our public facing documentation. We'll need to give folks a migration path away from naming the detail namespace before we change its name.

@chiphogg chiphogg closed this May 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename detail namespace to auimpl
1 participant