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 #32, Requirements cleanup and add MM2000.2 #37

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Aug 3, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
NA

Expected behavior changes
None, requirements updates only (cleanup and add missing)

System(s) tested on
NA

Additional context
Needs careful review... just submitting the changes to start the process

Third party code
None

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

@skliper skliper added this to the Draco milestone Aug 3, 2022
@skliper skliper requested a review from dmknutsen August 3, 2022 17:14
@dzbaker
Copy link
Contributor

dzbaker commented Aug 3, 2022

CCB 3 Aug 2022: Approved pending review.

@skliper
Copy link
Contributor Author

skliper commented Aug 5, 2022

Updates included here:
Just rationale updates for clarification: MM1007, MM1008
Grammatical/formatting (no intent change): MM1011, MM2000 MM2000.1, MM2002, MM2003, MM2003.2, MM2004, MM2100.1

Minor updates but no intent change:
MM2003.1 - removed <MISSION_DEFINED> qualifier on CRC
MM2004.1 - added context (no intent change)

New requirement (already in implementation):
MM2002.2 - verification of poke data size

@dzbaker - this is OK to merge.

@dzbaker dzbaker merged commit f8cf7fb into nasa:main Aug 8, 2022
Copy link
Contributor

@dmknutsen dmknutsen left a comment

Choose a reason for hiding this comment

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

Looks great!

@skliper skliper deleted the fix32-requirements_cleanup branch September 19, 2022 16:53
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.

Clarify incomplete or unclear requirements for commands
3 participants