-
Notifications
You must be signed in to change notification settings - Fork 631
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 upgrade instructions to README #687
Conversation
@gingerlime if you want to review |
README.rdoc
Outdated
{change was added in #647}[link:https://github.com/ctran/annotate_models/pull/647] to change this. You {can read more here}[https://github.com/ctran/annotate_models/issues/663]. | ||
|
||
There are a few ways of fixing this. | ||
1. If using CLI explicitly pass in models flag using +--models+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps the easiest way is to run rails g annotate:install
? :)
README.rdoc
Outdated
@@ -50,6 +50,18 @@ It also annotates geometrical columns, geom type and srid, when using | |||
Also, if you pass the -r option, it'll annotate routes.rb with the output of | |||
<code>rake routes</code>. | |||
|
|||
== Upgrading to 3.X and annotate models not working? | |||
In versions 2.7.X the annotate gem defaulted to annotating models if no arguments were passed in. The annotate gem by default would not allow for routes and models to be annotated together. A | |||
{change was added in #647}[link:https://github.com/ctran/annotate_models/pull/647] to change this. You {can read more here}[https://github.com/ctran/annotate_models/issues/663]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pinging me. Just made a couple of small suggestions :) |
@gingerlime added some changes, wanna take another look? |
just spotted one small thing (potentially). Otherwise 👍 Sorry if this is a dumb question, but couldn't you detect if |
@gingerlime sorry for the way delayed response, but yes can make that change. In the mean time will add this so folks can see this in the README. |
Updates the readme to include a note about upgrading the gem to version 3 and that it can default to doing nothing.
Updates the readme to include a note about upgrading the gem to version 3 and that it can default to doing nothing.
No description provided.