javascript-exercises icon indicating copy to clipboard operation
javascript-exercises copied to clipboard

Add disclaimer about not submitting personal solutions to PULL_REQUEST_TEMPLATE.md for all exercise repos

Open Asartea opened this issue 2 years ago • 15 comments

Complete the following REQUIRED checkboxes:

  • [x] I have thoroughly read and understand The Odin Project Contributing Guide
  • [x] The title of this issue follows the location for request: brief description of request format, e.g. Exercises: Add exercise on XYZ

The following checkbox is OPTIONAL:

  • [x] I would like to be assigned this issue to work on it

1. Description of the Feature Request: There are still regularly people who submit personal solutions to all of the various exercise repos. We do mention that those repos don't accept solutions in the README.md, but that line is unhighlighted, at the bottom, and easily missable. Furthermore due to the fork > clone > solve > upload to personal repo workflow that a learner is likely to use when solving the exercises, they will get a big yellow/orangeish banner above both their own repo and the forked repo with a call out to Contribute now. Adding an additional line somewhere in the PR template (<!-- WARNING: Do not submit personal solutions to this repo. Pull Requests containing personal solutions will be summarily closed and will not be merged --> ?) would provide another layer of catching a learners eyes before they submit a superflous PR

2. Acceptance Criteria:

  • [ ] Some form of warning is added to all PR templates in the various exercise repos

Asartea avatar Jul 21 '23 16:07 Asartea

Yeah I agree with this. We definitely need to make it more visible.

CouchofTomato avatar Jul 24 '23 15:07 CouchofTomato

What do you think about adding the line and also adding a bit of bolding in the bullet point?

Do not submit your solutions to this repo, as any PRs that do so will be closed without merging.

Asartea avatar Jul 24 '23 17:07 Asartea

Honestly we could probably move that verbiage to the opening paragraph:

These JavaScript exercises are intended to complement the JavaScript content on The Odin Project (TOP). They should only be done when instructed during the course of the curriculum. Please DO NOT submit your solutions to this repo, as any PRs that do so will be closed without merging.

Also wouldn't be opposed to adding a comment in the PR template. @Asartea did you still want to be assigned?

thatblindgeye avatar Feb 04 '24 18:02 thatblindgeye

@Asartea did you still want to be assigned?

yes

Asartea avatar Feb 05 '24 19:02 Asartea

@Asartea you got it. There is a similar issue linked above from our css exercise repo which should be taken into account. If you have any questions feel free to ping!

thatblindgeye avatar Feb 05 '24 19:02 thatblindgeye

@Asartea If you want to update the CSS one as well that is fine, or I will do it. Either way we should coordinate our language I think.

UniquePixels avatar Feb 05 '24 19:02 UniquePixels

I have gone ahead with the css repo: https://github.com/TheOdinProject/css-exercises/pull/493

UniquePixels avatar Feb 18 '24 22:02 UniquePixels

Sorry, I have been really busy and then really sick. Will give this a look now

Asartea avatar Feb 28 '24 15:02 Asartea

No worries! Hopefully you're feeling better, but take your time!

thatblindgeye avatar Feb 29 '24 13:02 thatblindgeye

@Asartea just an FYI I merged the PR in the css exercise repo. If we'd want to tweak any verbiage further in this js exercise readme we can do a followup for CSS.

thatblindgeye avatar Apr 20 '24 12:04 thatblindgeye