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 #41, Remove unrequired internal delete command handling #46

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Aug 3, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
CI

Expected behavior changes
No longer supports FM_DELETE_INT_CC or the related strange (out of family) logic. FM is operator interface to file management (even the design document says it doesn't do "automated" actions).

System(s) tested on
CI

Additional context
Could have deprecated, but can't find any external references to it so put in this PR to see if there is support for a direct removal. It didn't behave as described to begin with.

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
@dzbaker dzbaker merged commit 15f3b1c into nasa:main Aug 10, 2022
@skliper skliper deleted the fix41-rm_delete_int branch September 19, 2022 16:40
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.

Internal delete command support not in requirements, doesn't match description
3 participants