Uh oh!
There was an error while loading. Please reload this page.
Fix brand regex - #331
Conversation
conda-forge-linter
commented
Mar 20, 2020
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 ( I do have some suggestions for making it better though... For recipe:
|
isuruf
commented
Mar 20, 2020
@conda-forge-admin, rerender |
…nda-forge-pinning 2020.03.20
SylvainCorlay
commented
Mar 20, 2020
Even if this works, I think we should consider completely removing that patch. |
jjhelmus
commented
Mar 20, 2020
I'm fine removing this patch entirely. It was in conda-forge before I started working on the python recipe so I kept it. |
mingwandroid
commented
Mar 20, 2020
What about using the anaconda branding mechanism instead? We change the c code. I can template the patch and apply the correct replacement at build time if that's acceptable? Branding is kinda useful if you change things significantly and I think, for better or worse we do that. But I don't mind if we remove it entirely for conda-forge either really. |
jakirkham
commented
Mar 20, 2020
Having some kind of branding is useful I think. Though I don't know if we are discussing removing it or just some minor tweaks at this point. |
SylvainCorlay
commented
Mar 20, 2020
I don't know where to find that recipe? |
SylvainCorlay
commented
Mar 20, 2020
This PR does a minor tweak but I was suggesting we should consider removing it completely. |
SylvainCorlay
commented
Mar 22, 2020
For your information, I opened #332 completely removing the patching of |
SylvainCorlay
commented
Mar 23, 2020
Any interest in getting this in? |
isuruf
commented
Mar 23, 2020
Can you bump the build number? |
SylvainCorlay
commented
Mar 23, 2020
Done |
conda-forge-linter
commented
Mar 23, 2020
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 ( I do have some suggestions for making it better though... For recipe:
|
SylvainCorlay
commented
Mar 23, 2020
@isuruf I bumped the build number - and did some more testing locally. |
SylvainCorlay
commented
Mar 23, 2020
The rerender seems to be problematic. I don't see why there should be a |
isuruf
commented
Mar 23, 2020
One build in linux-64 passing is enough |
Fixes#194
I would really advocate from completely removing the
brand_python.pythough. I don't think that modifying_sys_versionis a clean way of changing the banner.Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)