-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add option to force Unix EOL on YAML export via the CLI #316
Conversation
I believe this is ready for review! |
thank you so much for the contribution, @kevmk04! 🙏 This is super cool. 🙌 |
@Vitor-Avila Should be all good now with your PR comments. |
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.
Thank you so much for working on this, @kevmk04! I left some minor comments around the tests. We should be good to merge this once they're addressed. 🙌
PR adds an optional argument to the CLI export commands to force Unix EOLs for the exported files. This is intended to address issues when exporting into Windows environments where we have multiple developers using different OS.