-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Extend to not just pkgs/by-name
#45
Comments
how would that look like? i have little idea of how the (i assume) relevant github APIs look like, nor of the |
@stuebinm Thanks for the interest! I just took the time to write it down in a high level here: #56. Turned out a bit long, but it's important context. I think the While we primarily work asynchronously via GitHub, we sometimes use the Nixpkgs Architecture Matrix room or my weekly office hours to synchronise. Would be great if you could join those! |
Recently there was an automated mass removal of
lib.mdDoc
which now requires subsequent PRs whenever a PR that introduces more instances gets merged.The approach used by nixpkgs-check-by-name makes sure that this problem doesn't happen. We should consider integrating such treewide removals to benefit from that. This will also be relevant for the upcoming automated
pkgs/by-name
migration.In particular, while this tool currently only has infrastructure for detecting when new instances of deprecated patterns are introduced, I've been planning to add infrastructure to also do automated replacements of such deprecated patterns.
For now I haven't gotten to that, but let's use this issue to track such developments.
The text was updated successfully, but these errors were encountered: