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

separating Ruby and ROR rubocop configs #29

Merged
merged 6 commits into from
Jan 15, 2020
Merged

Conversation

shloch
Copy link
Contributor

@shloch shloch commented Dec 13, 2019

  • created a ror/ folder and copied .rubocop.yml config file i worked on few days ago which was a little less restrictive(or more friendly)
  • also copied STICKLER from ruby/ folder into ror/ folder
  • removed from ruby/.rubocop.yml file configs specific to RAILS projects
  • crosschecked ror/.rubocop.yml file if there should be anything specific to ruby alone (it was OK)
  • edited general README, to include links to Ruby & Rails folders
  • edited heading of Ruby & Rails READMEs to have specific course-title

Copy link
Contributor

@nidalaa nidalaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job @shloch !
Thank you so much for this PR 🙇‍♀

Can you make small changes I requested in the comments?

P.S. We will wait with merge until corresponding changes in Pathwright are made :)

README.md Outdated
@@ -59,6 +60,7 @@ In that case check detailed instructions for each linter:

- [css](./css#troubleshooting)
- [ruby](./ruby#troubleshooting)
- [Ruby on Rails](./ror)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be ./ror#troubleshooting

- "bin/*"
- "db/**/*"
- "config/**/*"
- "Guardfile"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's leave Guardfile and Rakefile here please 🙏


# add the files here you want to be ignored by stylelint
files:
ignore:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please make sure that ignore contains the same files as exclude here https://github.com/microverseinc/linters-config/pull/29/files#diff-bb818e2d66f2dd3502ac21100eb9d755R2

@shloch
Copy link
Contributor Author

shloch commented Jan 3, 2020

hello @nidalaa
Thanks for the observations, Happy holidays
The above proposed observations has been incorporated

@nidalaa nidalaa merged commit 3238a8e into microverseinc:master Jan 15, 2020
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