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

Update to contribution guidelines #124

Merged
merged 11 commits into from
Mar 1, 2024
Merged

Update to contribution guidelines #124

merged 11 commits into from
Mar 1, 2024

Conversation

evan-masseau
Copy link
Contributor

@evan-masseau evan-masseau commented Feb 27, 2024

Description

Updates our contributing guide and issue templates per our discussions about streamlining and adding firmer direction about how to provide feedback.

Changelog / Code Overview

  • Updated contributing doc with better guidelines about opening issues, and firm language about expectations.
  • Linked to contributing guides in the issue templates, made important questions required
  • Fixed some incorrect/missing details about using the repo locally
  • Added precommit and CI validation to prevent npm usage by rejecting package-lock.json file
  • Close all discussion topics except Announcements (now directing people to issues)
  • Create labels for missing-details or off-topic issues

Related Issues/Tickets

CHNL-5893
CHNL-4970

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
@evan-masseau evan-masseau changed the title Prevent package-lock from getting committed Contributing updates Feb 27, 2024
@evan-masseau evan-masseau marked this pull request as ready for review February 27, 2024 17:00
@evan-masseau evan-masseau requested a review from a team as a code owner February 27, 2024 17:00
@evan-masseau evan-masseau requested review from ndurell and removed request for a team February 27, 2024 17:00
yarn.lock Outdated Show resolved Hide resolved
.github/CONTRIBUTING.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@evan-masseau evan-masseau changed the title Contributing updates Update to contribution guidelines Feb 29, 2024
validations:
required: true
- type: checkboxes
attributes:
label: Checklist
options:
- label: I have determined whether this bug is also reproducible in a vanilla project
- label: I have determined whether this bug is also reproducible in a vanilla project, such as the example app in this repository.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be overkill to add a checkbox for, i've read and understand the contributing policy or something to that effect? can be added to bugs and feature templates

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea

@evan-masseau evan-masseau merged commit c07647a into master Mar 1, 2024
5 checks passed
@evan-masseau evan-masseau deleted the ecm/contributing branch March 1, 2024 21:46
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.

3 participants