Skip to content

Integration Candidate 20200121 #54

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jan 27, 2020
Merged

Integration Candidate 20200121 #54

merged 4 commits into from
Jan 27, 2020

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 21, 2020

Describe the contribution
Fix #40, fix #52

Testing performed
Steps taken to test the contribution:

  1. Current CI passes
  2. Enhanced CI in GUI Command definitions use '--word' option unrecognized by cmdUtil #40 passes debug/with deprecated elements testing

Expected behavior changes

  • Now excepts --word as alias to --long
  • Fully converted to python3 and fixed tlm subscription

System(s) tested on

  • CI: Ubuntu 18.04, IC bundle

Additional context
N/A

Third party code
N/A

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added this to the 2.2.0 milestone Jan 21, 2020
@skliper
Copy link
Contributor Author

skliper commented Jan 27, 2020

CCB 20200122 - Reviewed and approved for merge to master

@skliper skliper added the CCB:Approved Indicates approval by CCB label Jan 27, 2020
@skliper skliper merged commit 343d8b3 into master Jan 27, 2020
@skliper skliper deleted the ic-20200121 branch January 27, 2020 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CCB:Approved Indicates approval by CCB
Projects
None yet
Milestone
Bootes
Development

Successfully merging this pull request may close these issues.

Further fixes for python3 and telemetry processing GUI Command definitions use '--word' option unrecognized by cmdUtil
2 participants
@skliper @dmknutsen