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 CONTRIBUTING.md to remove outdated information #1416

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

benjagm
Copy link
Collaborator

@benjagm benjagm commented Jun 20, 2023

Summary: This file is a brand new version of the contributing file to remove outdate information.

Resolves #1414

Summary: This file is a brand new version of the contributing file to remove outdate information.

Related issue: #1414
Copy link
Member

@Relequestual Relequestual left a comment

Choose a reason for hiding this comment

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

Made a few suggestions for change.

I think this would be easier to review by spreading the PR over multiple commits.

There are a lot of things that no longer apply, but we don't exactly have a replacememt process ready. Such as, we shouldn't mention drafts or milestones, because right now we aren't using them in any defined way.

This is a good start though for sure.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@gregsdennis
Copy link
Member

Pinging @benjagm (just a reminder).

Copy link
Collaborator Author

@benjagm benjagm left a comment

Choose a reason for hiding this comment

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

Looks good to me. I can't approve the PR with my permissions.

@gregsdennis
Copy link
Member

@benjagm it's your PR. I think @Relequestual's comments are for you to address. 😉

@benjagm
Copy link
Collaborator Author

benjagm commented Sep 14, 2023

@benjagm it's your PR

Thanks Greg! I totally forgot about this one 🤪

benjagm and others added 4 commits September 15, 2023 11:09
Co-authored-by: Ben Hutton <relequestual@gmail.com>
Co-authored-by: Ben Hutton <relequestual@gmail.com>
Co-authored-by: Ben Hutton <relequestual@gmail.com>
Adding suggestion made in the PR.
@benjagm
Copy link
Collaborator Author

benjagm commented Sep 15, 2023

Thanks everyone for the reviews and comments. I tried to address all the changes with the last PR.

Copy link
Member

@Relequestual Relequestual left a comment

Choose a reason for hiding this comment

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

Sorry for the delay. Looks all good! Thanks.

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.

Contributing guide file is outdated
3 participants