-
Notifications
You must be signed in to change notification settings - Fork 8
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
git-sync ignores whitespace diffs. #105
Conversation
@@ -4,8 +4,8 @@ channels: | |||
- http://ssb.stsci.edu/astroconda | |||
- defaults | |||
dependencies: | |||
#- bqplot-image-gl==1.4.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jaytmiller I'm not sure why these packages needed to be specified here, but they were necessary in the latest jwebbinar build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think packages in base.yml are only going into the base Python environment.
Aside from env cloning, packages to be installed in all Python environments are either defined in common-env or common-scripts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check into difference between base.yml and packages spec'ed in common-env or common-scripts.
@@ -4,8 +4,8 @@ channels: | |||
- http://ssb.stsci.edu/astroconda | |||
- defaults | |||
dependencies: | |||
#- bqplot-image-gl==1.4.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think packages in base.yml are only going into the base Python environment.
Aside from env cloning, packages to be installed in all Python environments are either defined in common-env or common-scripts.
We were having a problem where diffs were being detected (technically, properly), but the only differences were whitespace. This was resulting in
git-sync
creating backups of the "changed" files - an unwanted behavior. Changes in this PR adjust some logic, and add a flag togit diff
to ignore whitespace.Also fixed logic error that evaluates
$USE_CENTRAL_ECR