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

my proposed updates, part 2 #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

frankgard
Copy link

I didn't find subdir "functional" within "test", but "controllers". So, I think this is a typo. In addition, the usage of "rails" and "rake" (especially when to use which) is not trivial, is it?

Cheers,
Frank.

P.S.: Sorry for any inconvenience due to unnecessary micro-commits. It was because I used Rails 4 by accident and already had pushed them when I recognized that.

Frank Gard added 8 commits March 7, 2016 18:32
"rails db:migrate" won't work.
… instead of "rails test" which won't work.
controller tests are not located in "test/functional".
This reverts commit ecbf25bad0a83550824e0bbc4f4c383b851ba57d.

When using Rails 5.0, "rails test" will work! So, my changes
were outdated.
This reverts commit 6213a8ba3b10de56a3cfe9c806eda2da22375c78.

When using Rails 5.0, "rails db:migrate" will work! So, my changes
were outdated.
… instead of "rake test". I see, I'm not the only one
whose brain is "version 4.2" :).
… instead of "rake generate scaffold".
While "rake test" and "rails test" both are possible,
"rake generate scaffold …" is not a valid syntax IMHO.
There is no obvious reason for using "rake-stats"
(instead of "rails-stats") as link-target, is it?
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.

1 participant