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 Optional Path for Rubocop #676

Merged
merged 4 commits into from
Nov 23, 2015
Merged

Add Optional Path for Rubocop #676

merged 4 commits into from
Nov 23, 2015

Conversation

emanuelet
Copy link
Contributor

add option to specify the path for rubocop in the settings.
The relative beautifier now retrieve the path similarly to how the php-cs-fixer one does.

I tested it and is working correctly on my system.

@emanuelet
Copy link
Contributor Author

apparently the appveyor build is failing generating documentation, or so it seems. Is it safe to ignore?

@Glavin001
Copy link
Owner

I will ignore the AppVeyor tests.

Could you run "coffee docs/" to build the documentation with the new option and commit that change with this pull request? Looks about ready to merge. Thanks!

@emanuelet
Copy link
Contributor Author

done

@Glavin001
Copy link
Owner

Awesome. Great work! Thank you so much for contributing!

Glavin001 added a commit that referenced this pull request Nov 23, 2015
Add Optional Path for Rubocop
@Glavin001 Glavin001 merged commit baad5e2 into Glavin001:master Nov 23, 2015
@emanuelet
Copy link
Contributor Author

my pleasure

@emanuelet
Copy link
Contributor Author

hi, can I ask you if you can push a new minor release?

@Glavin001
Copy link
Owner

Published to v0.28.20
Thanks

@emanuelet
Copy link
Contributor Author

thank you very much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants