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

Apply code ordering as recommended by Thoughtbot Rails style guide #580

Closed
wants to merge 1 commit into from
Closed

Apply code ordering as recommended by Thoughtbot Rails style guide #580

wants to merge 1 commit into from

Conversation

hmallett
Copy link
Contributor

@hmallett hmallett commented Feb 9, 2015

Specifically apply the following rules:

  • Order ActiveRecord associations alphabetically by attribute name.
  • Order ActiveRecord validations alphabetically by attribute name.
  • Order ActiveRecord associations above ActiveRecord validations.

https://github.com/thoughtbot/guides/tree/master/style/rails

Specifically apply the following rules:
- Order ActiveRecord associations alphabetically by attribute name.
- Order ActiveRecord validations alphabetically by attribute name.
- Order ActiveRecord associations above ActiveRecord validations.

https://github.com/thoughtbot/guides/tree/master/style/rails
@mvz
Copy link
Member

mvz commented Feb 18, 2015

I like this. Is there any way we can automate this?

@hmallett
Copy link
Contributor Author

I don't know of one. :(

@mvz
Copy link
Member

mvz commented Jan 28, 2018

Closing this due to merge conflicts.

@mvz mvz closed this Jan 28, 2018
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.

2 participants