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.

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 a language header to non-English feature files #17

Merged
merged 2 commits into from
Dec 11, 2019

Conversation

rayjolt
Copy link
Contributor

@rayjolt rayjolt commented Dec 10, 2019

This pull request adds language headers to non-English feature files. Fixes #15.

Ignore the .idea folder, used by IntelliJ IDEs like PyCharm. Some of it
can be stored in version control, but it is probably best to leave
editor settings to individual developers.
@rayjolt
Copy link
Contributor Author

rayjolt commented Dec 10, 2019

Will try and figure out the reason for these failures tomorrow.

@rayjolt rayjolt force-pushed the language-header branch 2 times, most recently from 5841da3 to 0044105 Compare December 10, 2019 23:09
@ducminh-phan
Copy link
Owner

@rayjolt I have fixed the issue with Codacy. Could you add a test to cover the case there is no feature in the file?

Language headers are necessary for non-English feature files, as without
them the non-English keywords are not recognised as keywords.

Fixes ducminh-phan#15
@rayjolt
Copy link
Contributor Author

rayjolt commented Dec 11, 2019

Thanks for fixing the Codacy issue. I was staring at that for quite a while trying to figure out what was wrong. :P

@ducminh-phan ducminh-phan changed the base branch from master to develop December 11, 2019 13:32
@ducminh-phan
Copy link
Owner

Everything seems good! Thank you for your contribution. I'm going to release v1.0.1 soon.

@ducminh-phan ducminh-phan merged commit ee27d54 into ducminh-phan:develop Dec 11, 2019
@rayjolt
Copy link
Contributor Author

rayjolt commented Dec 11, 2019

Thank you!

@rayjolt rayjolt deleted the language-header branch December 11, 2019 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants