-
-
Notifications
You must be signed in to change notification settings - Fork 240
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
docs(contribution): Windows based contributors developer workflow #999
Conversation
Modern IDEs support (at last!) LF line endings. This commit ensures that - Every text file is checked out with LF line endings in the working directory (eol=lf) - Every potentially introduced CRLF in a text file will be converted back to LF on staging (* text=auto) cf. https://git-scm.com/docs/gitattributes
- Editorconfig ensures the defined code styles is taken into account - Jest runner helps troubleshoot failing tests
@philsturgeon @P0lip In order to have a working Thoughts? |
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.
@philsturgeon do you have any concerns or questions? If not, let's merge it. |
Checklist
Does this PR introduce a breaking change?
Yes
No
Adds recommended vscode extensions
Adds a preconfigured Linux/node12 based development container for troubleshooting purposes
Provides some insights to contributing to Spectral from a Windows based environment
Note: The changes to the
.gitattributes
files are mostly related to the container based development process where the files will be certainly checked out in a Windows based context then "shared" between the two OSes (and the fact that Linux doesn't like CRLFs at all).