You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The wiki has its own Contributing Guide section which has links to an archived discord channel as a place to seek help with contributing. There's more improvements to be made in my mind, like changing the line:
Install the app on your local machine.
to:
Clone the repo onto your local machine.
There's some consistency issues with using punctuation in the list (some end with, some don't). This is what I'd like to do:
Make sure the .github version of CONTRIBUTING.MD is up-to-date with this repo's.
Remove the repo-specific CONTRIBUTING.MD and let the .github one be the one source of truth.
Remove the mention of running tests from CONTRIBUTING.MD and instead rely on contributors following the Wiki for steps they need to do.
Update various dead links, inconsistent punctuation/inclusion of $ in commands, change all rails commands to bin/rails, use the bin/test script for running tests - especially because that script runs erblint but the instructions fail to mention it needs to be run currently, bin/dev for running the app, perhaps look into improving bin/setup script so the contributor can just run that instead of going step-by-step with installing the gems, JS dependencies, creating the DB, running the tests before starting work, seeding & populating the DB.
Include the wiki article on squashing commits in the Contributing Guide section of the wiki.
Looks like there was a regression of #4323 brought on by #4478 -- the CONTRIBUTING.md for this repo now says to run yarn test again, which should absolutely change. Maybe there can be a new page on the wiki about commands to run before submitting a PR?
The team settled on having separate CONTRIBUTING docs for handling repo specific concerns, so I wouldn't want to consolidate everything into the parent CONTRIBUTING doc. But other than that all of your ideas seem good here. Ditching the $ in the bash commands is a great idea to give a better copy/paste experience.
You still interested in making a PR (or PRs) for this? @scheals
Glad to answer any questions you have or give guidance on things if you want.
Hey @JoshDevHub, I am still interested albeit I am going to focus on the RuboCop lesson at the moment. After I'm done with that I am going to have to set up TOP locally once again from scratch, so then my experience will be fresh.
If someone gets to this before I can (which, if everything goes according to the plan should be around start of may) they should go ahead.
For sure whoever wants can change the testing bit right now could do so, that seems like a quick and important fix. Same with removing $s.
I wonder why the current Gemfile setting for Ruby version has not stopped joel from discord from running stuff. I seem to recall that I was trying to run someone's Rails app recently and I was stopped in my tracks because of a Gemfile-defined Ruby version being different from the one I had enabled (and, I didn't have the target Ruby version at all, in fact). Bundler docs seem to suggest that the current way to link .ruby-version with Gemfile is:
rubyfile: ".ruby-version"
Aaand actually, scratch that, I just googled around and supposedly Heroku does not support this and the way it is implemented here should work. So what am I missing? bin/test does use bundle exec @_@
Checks
Feature Request: brief description of feature request
format, e.g.Feature Request: Add a dark mode to the website
Description of the Feature Request
Currently, there are two versions of
CONTRIBUTING.MD
:Haven't looked into them too deep but they seem to have diverged according to their commit histories - and have for sure diverged after I PR'd that
yarn test
change Fix: remove jest testing from app PR instructions #4323The wiki has its own Contributing Guide section which has links to an archived discord channel as a place to seek help with contributing. There's more improvements to be made in my mind, like changing the line:
to:
There's some consistency issues with using punctuation in the list (some end with, some don't). This is what I'd like to do:
.github
version ofCONTRIBUTING.MD
is up-to-date with this repo's.CONTRIBUTING.MD
and let the.github
one be the one source of truth.CONTRIBUTING.MD
and instead rely on contributors following the Wiki for steps they need to do.$
in commands, change allrails
commands tobin/rails
, use thebin/test
script for running tests - especially because that script runserblint
but the instructions fail to mention it needs to be run currently,bin/dev
for running the app, perhaps look into improvingbin/setup
script so the contributor can just run that instead of going step-by-step with installing the gems, JS dependencies, creating the DB, running the tests before starting work, seeding & populating the DB.Contributing Guide
section of the wiki.Acceptance criteria
.github
'sCONTRIBUTING.MD
with the changes in this repo'sCONTRIBUTING.md
CONTRIBUTING.MD
CONTRIBUTING.MD
Additional Comments
No response
The text was updated successfully, but these errors were encountered: