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

(#806) Use ASCII quotes instead of Unicode quotes #807

Merged
merged 2 commits into from
Nov 22, 2019

Conversation

rodjek
Copy link
Contributor

@rodjek rodjek commented Nov 22, 2019

Added a custom RuboCop cop to detect any smart quotes that sneak in in the future. It can automatically fix smart single quotes, fixing double quotes can be implemented if needed in the future.

Fixes #806

@rodjek rodjek requested a review from a team as a code owner November 22, 2019 00:00
@rodjek rodjek added this to the December 2019 milestone Nov 22, 2019
@coveralls
Copy link

coveralls commented Nov 22, 2019

Coverage Status

Coverage remained the same at 91.79% when pulling 4748aee on rodjek:gh-806 into 8e4bcf8 on puppetlabs:master.

@glennsarti
Copy link
Contributor

I'm kinda surprised there isn't a rubocop plugin for smart-quotes to be honest

@glennsarti glennsarti merged commit 8283205 into puppetlabs:master Nov 22, 2019
@rodjek
Copy link
Contributor Author

rodjek commented Nov 22, 2019

Yeah, same. I had a search around on rubygems.org but couldn't find one 🤷‍♂️

@rodjek rodjek deleted the gh-806 branch November 22, 2019 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(UI) Remove UTF-8 "smart quotes" from PDK output
5 participants