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 contributing #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

update to contributing #13

wants to merge 1 commit into from

Conversation

DavidPHirsch
Copy link
Contributor

Signed-off-by: David Hirsch [email protected]

This PR

adding details to the contributing doc

Signed-off-by: David Hirsch <[email protected]>

Signed-off-by: David Hirsch <[email protected]>

We hold regular meetings which you can see [here](https://github.com/open-feature/community/#meetings-and-events).
* [New Contributor Guide](#contributing-guide)
Copy link
Member

Choose a reason for hiding this comment

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

This looks out of alignment

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what do you mean?

Copy link
Member

Choose a reason for hiding this comment

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

I don't think this anchor exists.


We hold regular meetings which you can see [here](https://github.com/open-feature/community/#meetings-and-events).
* [New Contributor Guide](#contributing-guide)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this anchor exists.

* [Pull Request Lifecycle](#pull-request-lifecycle)
* [Development Environment Setup](#development-environment-setup)
* [Sign Your Commits](#sign-your-commits)
* [Pull Request Checklist](#pull-request-checklist)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this anchor exists.


⚠️ **Explain how to set up a development environment**

## Sign Your Commits
Copy link
Member

Choose a reason for hiding this comment

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

Do you mean sign-off on your commits? Signed commits are different.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

* The original github issue
* Our Slack channel

## Pull Request Lifecycle (TODO)
Copy link
Member

Choose a reason for hiding this comment

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

Please add the PR lifecycle doc or remove this section.


⚠️ **Explain your pull request process**

## Development Environment Setup (TODO)
Copy link
Member

Choose a reason for hiding this comment

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

You should likely remove this section. Configuring a development environment will vary wildly based on the repo.

@beeme1mr beeme1mr removed the request for review from oleg-nenashev January 3, 2023 20:14
@beeme1mr
Copy link
Member

@DavidPHirsch bump :)

@beeme1mr
Copy link
Member

@DavidPHirsch another bump 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants