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

Fix linting issues in specs #465

Closed
ankitkataria opened this issue Oct 28, 2019 · 2 comments · Fixed by #469
Closed

Fix linting issues in specs #465

ankitkataria opened this issue Oct 28, 2019 · 2 comments · Fixed by #469

Comments

@ankitkataria
Copy link
Member

Describe the bug
Some of the specs were added before rubocop was added for linting. As a result some of them were written without linting checks. These linting errors needs to be fixed. Check rubocop to see how linting works.

Linting fixes are required in following directories inside ./spec

Separate PRs for fixing linting issues of a group of files are expected. Let us know which files you are working on by commenting below.

To Reproduce
Run bundle exec rubocop ./spec/

Expected behavior
Should show no offences

Are you working on this issue? (Yes/No)
No

@YashKumarVerma
Copy link
Member

@tachyons, there are linting issues in other files as well, (as of now, the count is at 44k). Will PR addressing those be welcomed?

@ciphereck
Copy link
Contributor

@ankitkataria @tachyons I ran bundle exec rubocop ./spec/ again, and it's still showing 12 offenses in the latest master branch. Can we reopen this issue?

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 a pull request may close this issue.

4 participants