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.

Sign up for GitHub

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

Jump to bottom

my proposed updates, part 2 #2

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

my proposed updates, part 2 #2

wants to merge 8 commits into from

Commits on Mar 7, 2016

  1. Use rake instead of rails to migrate 

    "rails db:migrate" won't work.
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    6213a8b View commit details
    Browse the repository at this point in the history

  2. Use "rake test" for testing 

    … instead of "rails test" which won't work.
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    ecbf25b View commit details
    Browse the repository at this point in the history

  3. correct controller tests' path 

    controller tests are not located in "test/functional".
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    e15f76d View commit details
    Browse the repository at this point in the history

  4. Revert "Use "rake test" for testing" 

    This reverts commit ecbf25bad0a83550824e0bbc4f4c383b851ba57d.

When using Rails 5.0, "rails test" will work! So, my changes
were outdated.
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    7d96243 View commit details
    Browse the repository at this point in the history

  5. Revert "Use rake instead of rails to migrate" 

    This reverts commit 6213a8ba3b10de56a3cfe9c806eda2da22375c78.

When using Rails 5.0, "rails db:migrate" will work! So, my changes
were outdated.
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    afce208 View commit details
    Browse the repository at this point in the history

  6. Use "rails test" 

    … instead of "rake test". I see, I'm not the only one
whose brain is "version 4.2" :).
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    77384d3 View commit details
    Browse the repository at this point in the history

  7. Use "rails generate scaffold …" 

    … instead of "rake generate scaffold".
While "rake test" and "rails test" both are possible,
"rake generate scaffold …" is not a valid syntax IMHO.
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    e3c7f19 View commit details
    Browse the repository at this point in the history

  8. Substitute "rake-stats" by "rails-stats" 

    There is no obvious reason for using "rake-stats"
(instead of "rails-stats") as link-target, is it?
    Frank Gard committed Mar 7, 2016
    Configuration menu
    Copy the full SHA
    db141e2 View commit details
    Browse the repository at this point in the history