NASA-AMMOS / aerie-ui

The client application for Aerie.
https://nasa-ammos.github.io/aerie-docs/
MIT License
28 stars 4 forks source link

Don't enforce SeqN directive order #1271

Closed cartermak closed 1 month ago

cartermak commented 1 month ago

Checked for duplicates

Yes - I've already checked

Alternatives considered

Yes - and alternatives don't suffice

Related problems

There isn't an intuitive reason (that I see) to enforce that directives are in a certain order, and the error messages aren't helpful because the grammar gets confused.

Describe the feature request

Suggest updating the SeqN grammar to allow directives in any order. If there are mission-specific preferences, it's trivial to implement custom linting to flag an error. Thoughts?

joswig commented 1 month ago

Fixed in #1275

https://github.com/NASA-AMMOS/aerie-ui/pull/1275/commits/01cedbe58a9366364d84cebdf480d11629cc86ae