-
Notifications
You must be signed in to change notification settings - Fork 658
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
Core: Default YAML header updates #2723
Core: Default YAML header updates #2723
Conversation
Cleanup
Added a "that" to remove an ambiguity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small nitpick, but otherwise good.
Combined the two sentences into one, per suggestion Co-authored-by: Nicholas Saylor <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
* Cleaning up (ArchipelagoMW#4) Cleanup * Adressed change about spaces no longer being replaced to underscores. Added a "that" to remove an ambiguity * Update data/options.yaml Combined the two sentences into one, per suggestion Co-authored-by: Nicholas Saylor <[email protected]> --------- Co-authored-by: Nicholas Saylor <[email protected]>
* Cleaning up (ArchipelagoMW#4) Cleanup * Adressed change about spaces no longer being replaced to underscores. Added a "that" to remove an ambiguity * Update data/options.yaml Combined the two sentences into one, per suggestion Co-authored-by: Nicholas Saylor <[email protected]> --------- Co-authored-by: Nicholas Saylor <[email protected]>
* Cleaning up (#4) Cleanup * Adressed change about spaces no longer being replaced to underscores. Added a "that" to remove an ambiguity * Update data/options.yaml Combined the two sentences into one, per suggestion Co-authored-by: Nicholas Saylor <[email protected]> --------- Co-authored-by: Nicholas Saylor <[email protected]>
* Cleaning up (#4) Cleanup * Adressed change about spaces no longer being replaced to underscores. Added a "that" to remove an ambiguity * Update data/options.yaml Combined the two sentences into one, per suggestion Co-authored-by: Nicholas Saylor <[email protected]> --------- Co-authored-by: Nicholas Saylor <[email protected]>
What is this fixing or adding?
Updating the default header of yamls to adress the fact spaces are no longer replaced to underscores.
How was this tested?
Reading