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 Document: contributor guide and Code of Conduct #169

Merged
merged 1 commit into from
Mar 2, 2022

Conversation

chenhong231
Copy link
Contributor

What is the purpose of the change
add contributor guide document

Brief changelog
new file: .github/PULL_REQUEST_TEMPLATE.md
new file: CODE_OF_CONDUCT.md
new file: CONTRIBUTING.md
modified: README.md
new file: WORKFLOW.md

Verifying this change
NONE

there are 2 things need to confirm:
PULL_REQUEST_TEMPLATE.md--------TODO: need to discuss the following step for crane project: basic checks pass,unit-test,integration-test
CONTRIBUTING.md -------------- TODO: need to discuss the committer nomination process.

@chenhong231
Copy link
Contributor Author

update the contributor guide

CONTRIBUTING.md Outdated
You can contribute to Crane in several ways. Here are some examples:

* Contribute to the Crane codebase.
* Contribute to the [Crane docs](https://github.com/gocrane/crane).
Copy link
Member

@qmhu qmhu Mar 2, 2022

Choose a reason for hiding this comment

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

Since crane don't have document/web repo now ,we don't mention it.

CONTRIBUTING.md Outdated

### Contribute to the Crane codebase

We encourage new comers actively join in Crane projects. The following paragraph introduce how to contribute in Crane way.
Copy link
Member

Choose a reason for hiding this comment

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

We need a Develop Guide here to help contributor know how to setup local env, prepare golang, docker and so on.
You can introduce some like this in this section.

CONTRIBUTING.md Outdated

### Contribute to the Crane codebase

We encourage new comers actively join in Crane projects. The following paragraph introduce how to contribute in Crane way.
Copy link
Member

Choose a reason for hiding this comment

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

comers?

CONTRIBUTING.md Outdated

#### Begin your contribute

If you find a typo in document, find a bug in code, or want new features, or want to give suggestions, you can [open an issue on GitHub](https://github.com/gocrane/crane/issues/new) to report it.
Copy link
Member

Choose a reason for hiding this comment

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

how about change Contribute to the Crane codebase to Develop Guide.
You can rename workflow.MD for detail

@@ -393,3 +393,9 @@ Something you should know about HPA recommendation:
## RoadMap
Please see [this document](./docs/roadmaps/roadmap-1h-2022.md) to learn more.

## Contributing
Copy link
Member

Choose a reason for hiding this comment

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

Please also add Contributing and Code of Conduct in navigation.

@chenhong231
Copy link
Contributor Author

done

@chenhong231
Copy link
Contributor Author

update document and squash commit.

@qmhu
Copy link
Member

qmhu commented Mar 2, 2022

/LGTM

@@ -25,6 +25,8 @@ Crane (FinOps Crane) is a cloud native open source project which manages cloud r
- [Analytics and Recommend Pod Resources](#analytics-and-recommend-pod-resources)
- [Analytics and Recommend HPA](#analytics-and-recommend-hpa)
- [RoadMap](#roadmap)
- [Contributing](#Contributing)
Copy link
Contributor

Choose a reason for hiding this comment

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

use contributing instead of Contributing to keep same style with other items?

@qmhu qmhu changed the title [ISSUE #162] add contributor guide document Update Document: contributor guide and Code of Conduct Mar 2, 2022
@qmhu qmhu merged commit cec0a4e into gocrane:main Mar 2, 2022
@qmhu qmhu linked an issue Mar 8, 2022 that may be closed by this pull request
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.

document for contributor guide
3 participants