-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rebuild for libxcb115 migration #7
Rebuild for libxcb115 migration #7
Conversation
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 ( |
…nda-forge-pinning 2023.05.21.19.48.32
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
PS. I think that your conda_forge_config.yaml is messing with the migrator. I think it will become more and more important as qt6 gains importantce. If you want to, I can help setup a qt5 branch. you would have to maintain 2 branches, but the bot would help. |
@hmaarrfk Qt5 branch existed in the past but I set up this to have only branch to maintain, as the only difference between Qt5 and Qt6 packages is the Qt dependency. Each update, even from the bot, should be merged twice and the bot added PR only for main branch (Qt6). Isn't there a way to maintain only one branch? |
I had a look deeper and I understand that my conda_forge_config.yaml works well for building but the migrator bot can't find this repo to open a PR because of the conditional |
yes |
I wonder if replacing these lines in meta.yaml would solve the problem:
To this
|
maybe.... i guess i feel like you are being "creative" here on this particular thing and it is likely not worth it. these conda-smithy bugs are really hard to track down due to the fact that they are often hard to test for. |
Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #6.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.