Skip to content
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

Fix CALACS version reporting #76

Merged
merged 1 commit into from
Apr 24, 2017
Merged

Conversation

pllim
Copy link
Contributor

@pllim pllim commented Apr 11, 2017

calacs.e --version now only reports the number. Added calacs.e -r to report version number and date.

Fix #75

c/c @jamienoss

@pllim pllim added this to the HSTDP-2017.2 milestone Apr 11, 2017
@pllim pllim requested a review from sosey April 11, 2017 20:18
Copy link
Member

@sosey sosey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. calwf3 is the only one adding the extra "current version" text, I've asked DP what they prefer and will update when I hear back

update the syntax with the options as well

--version now only reports the number. Added -r to report version number and date.
@pllim
Copy link
Contributor Author

pllim commented Apr 11, 2017

Okay. I updated the syntax printing.

$ calacs.e --version
9.1.0

$ calacs.e -r
Current version: 9.1.0 (21-Feb-2017)

$ calacs.e 
CALACS Version 9.1.0
syntax:  calacs.e [-t] [-s] [-v] [-q] [-r] [-1] input 

@sosey
Copy link
Member

sosey commented Apr 24, 2017

I'm merging this now @pllim

@sosey sosey merged commit f741c50 into spacetelescope:master Apr 24, 2017
@pllim pllim deleted the fix-ver-str branch April 24, 2017 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants