Codecov Report

Merging #2431 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2431   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files        1294     1294           
  Lines       30172    30172           
=======================================
  Hits        29775    29775           
  Misses        397      397

Hi @seanprashad, thanks so much for helping out with this!

We probably failed to explain the issue correctly. We need to update the documentation for tasks inside generated applications. The tasks run from the decidim root folder still need to be run via rake, since the decidim repo itself is not a Rails application and thus the rails command is not available.

That all means the only change we actually want to keep would be the one in decidim-dev/README.md, and that one should fix this issue. Could you update the PR? 😃

@seanprashad you are more than welcome to move to a more challenging PR next 😄

@deivid-rodriguez PR has been updated! I reverted back the changes in everything but decidim-dev/README.md 😁

@seanprashad Almost there but not quite yet! Changes in README.md need to be reverted since bundle exec rails s is correct there. Note that we cd into the application folder before that so those happen in the application folder (and bundle exec rake s does not exist anyways :))

@deivid-rodriguez I've updated the PR to fix the other lines that I modified which weren't supposed to be touched 😀 ! Please let me know if there is anything else I can do

Nothing else, thanks!!