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.

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

Audit "required" fields across the spec #4

Closed
getorymckeag opened this issue Dec 8, 2023 · 1 comment · Fixed by #91
Closed

Audit "required" fields across the spec #4

getorymckeag opened this issue Dec 8, 2023 · 1 comment · Fixed by #91
Assignees
@getorymckeag

Comments

@getorymckeag
Copy link
Contributor

Many recent changes to TROLIE have been missing marked "required" fields in the specification for inputs. This should be cleaned up to help provide better validation of the structures.
@getorymckeag getorymckeag self-assigned this Dec 8, 2023
@getorymckeag getorymckeag mentioned this issue Dec 8, 2023
Merged
@caindy caindy added this to the Forecast Proposal Spec final milestone Feb 14, 2024
@catkins-miso
Copy link
Contributor

I think we can remove this from the milestone, too.

@catkins-miso catkins-miso mentioned this issue Mar 14, 2024
Closed
7 tasks
@catkins-miso catkins-miso removed this from the Forecast Proposal Spec final milestone Mar 14, 2024
@getorymckeag getorymckeag mentioned this issue Mar 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@getorymckeag getorymckeag

Labels
None yet
Projects
TROLIE 1.0
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Minor cleanup items trolie/spec
3 participants
@caindy @catkins-miso @getorymckeag