-
-
Notifications
You must be signed in to change notification settings - Fork 561
Add check for migrator filename uniqueness #7691
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
base: main
Are you sure you want to change the base?
Conversation
numpy is using python314t.yaml already
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
|
See my comment on the smithy issue. GH search only covers up to 4000 feedstocks. So this approach won't work. |
what you quoted is that it will find only up to 4000 repos. But we only need the search to return a single hit for things to fail. The limit is not relevant for this detection. |
|
Well... In principle it works (even better than I expected, whole check completes in <10sec). The only big problem I'm running into is that none of Github's APIs actually seems to expose the "new" search that powers the results from the web UI (which returns the correct result). I've tried working around this in a couple of ways, but short of parsing the HTML after querying the web UI, I don't see how to run that search correctly. On the one hand I'd have been fairly optimistic that github will add this eventually, but OTOH, people have been asking for graphql integration of code search for at least ~8 years with no success, which doesn't exactly inspire confidence 😑 It's surprisingly hard to find out the current status about potential plans for the better search API to be exposed programmatically. |
|
I am almost 100% sure the text implies github won't search all of our repositories. It says
They are not limiting the results, but instead the number of repos searched. |
|
repost of my comment on another thread. |
You left out the following sentence which makes it clear (to me at least) that this is about search results. "The REST API will find up to 4,000 repositories that match your filters and return results from those repositories." Their API even provides a response whether the search results are complete I strongly doubt that they can even technically limit the number of repos they search, because it's all based on a giant index, and not individually walking the repos. |
This is WIP; see conda-forge/conda-smithy#2369