-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Use POSIX scripts for de-/activation scripts #313
Use POSIX scripts for de-/activation scripts #313
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 ( |
Can't we use shellcheck:
enabled: true in conda-forge.yml? |
I wasn't aware of that option to be honest. |
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
1ec2778
to
3762194
Compare
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 ( |
Maybe that should be added as an option. Can you open an issue in conda-smithy? As for this PR, I'd rather have this test with |
Isn't the shellcheck pretty orthogonal to this fix? :) |
That's a question on whether one sees regression tests as orthogonal to fixes or not ;). |
|
e833407
to
3ff5fc0
Compare
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
3ff5fc0
to
ec2164d
Compare
…ivation-scripts Use POSIX scripts for de-/activation scripts
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)Fixes gh-221