-
Notifications
You must be signed in to change notification settings - Fork 42
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #59 from permitio/perpare-to-publish
Clean Repo to Release
- Loading branch information
Showing
19 changed files
with
541 additions
and
234 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# Permit CLI Community Code of Conduct | ||
Permit CLI follows the [CNCF Code of Conduct](https://github.com/cncf/foundation/blob/master/code-of-conduct.md). | ||
|
||
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting | ||
the maintainers via <conduct@permit.io>. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,81 @@ | ||
# Contributing Guide | ||
|
||
We would love for you to contribute to this project and help make it even better than it is today! 💎 | ||
|
||
As a contributor, here are the guidelines we would like you to follow: | ||
- [New Commands Guidelines](#new-command-guidelines) | ||
- [Code of Conduct](https://github.com/permitio/permit-cli/blob/master/CODE_OF_CONDUCT.md) | ||
- [Question or Problem?](#question) | ||
- [Issues and Bugs](#issue) | ||
- [Feature Requests](#feature) | ||
|
||
|
||
|
||
## New Command Guidelines | ||
|
||
We are excited to have you onboard as a contributor to Permit CLI! 🎉 | ||
|
||
For new commands, we have a few guidelines to ensure consistency and maintainability: | ||
|
||
|
||
## <a name="question"></a> Got a Question or Problem? | ||
Come talk to us about Permit CLI, or authorization in general - we would love to hear from you ❤️ | ||
|
||
You can: | ||
- Raise questions in our [GitHub discussions](https://github.com/permitio/permit-cli/discussions) | ||
- Report issues and ask for features in [GitHub issues](https://github.com/permitio/permit-cli/issues) | ||
- Follow us on [Twitter](https://twitter.com/permit_io) to get the latest Permit CLI updates | ||
- Join our [Slack community](https://io.permit.io/slack) to chat about authorization, open-source, realtime communication, tech or anything else! We are super available on Slack ;) | ||
|
||
If you are using our project, please consider giving us a ⭐️ | ||
|
||
### <a name="issue"></a> Found a Bug? | ||
If you find a bug in the source code, you can help us by [submitting an issue](https://github.com/permitio/permit-cli/issues) or even better, you can [submit a Pull Request](#submit-pr) with a fix. | ||
|
||
Before you submit an issue, please search the issue tracker; maybe an issue for your problem already exists, and the discussion might inform you of workarounds readily available. | ||
|
||
We want to fix all the issues as soon as possible, but before fixing a bug, we need to reproduce and confirm it. | ||
In order to reproduce bugs, we require that you provide a full loom or other kind of terminal recording, so we can understand the issue and reproduce it. | ||
|
||
### <a name="feature"></a> Missing a Command / Feature? | ||
You can *request* a new feature by [submitting an issue](https://github.com/permitio/permit-cli/issues) to our GitHub Repository. | ||
Please provide as much detail and context as possible, along with examples or references to similar features, as this will help us understand your request better. | ||
|
||
We encourage you to contribute to Permit CLI by submitting a [Pull Request](#submit-pr) with your feature implementation and are happy to guide you through the process. | ||
|
||
We are always looking to improve Permit CLI and would love to hear your ideas! | ||
|
||
### <a name="submit-pr"></a> Submitting a Pull Request (PR) | ||
|
||
Pull requests are welcome! 🙏 | ||
|
||
Please follow these guidelines: | ||
|
||
1. **Create an Issue**: Open a [GitHub Issue](https://github.com/permitio/permit-cli/issues) for your feature or bug fix. | ||
2. **Fork & Branch**: Fork this repository and create a new branch based on `main`. Name your branch descriptively (e.g., `fix/issue-123`, `feature/new-fetch-provider`). | ||
3. **Write Tests**: If your changes affect functionality, include tests. | ||
4. **Update Documentation**: Ensure any new features or configurations are documented. | ||
5. **Check Quality**: Run all tests and linters: | ||
```bash | ||
npm run lint | ||
npm run test | ||
``` | ||
6. **Submit PR**: Open a pull request, linking to the issue and explaining your changes clearly. | ||
|
||
We aim to review all PRs promptly. After you submit a PR, here’s what you can expect: | ||
1. **Initial Review:** A maintainer will review your PR within a few days. If there are any issues, they will provide feedback. | ||
2. **Feedback:** If changes are needed, please make the necessary updates and push them to your branch. The PR will be updated automatically. | ||
3. **Approval:** Once your PR is approved, it will be merged into the main branch. | ||
4. **Release:** Your changes will be included in the next release of Permit CLI. We will update the changelog and release notes accordingly. | ||
5. **Announcement:** We will announce your contribution in our community channels and give you a shoutout! 🎉 | ||
|
||
### Contributor Checklist | ||
|
||
Before submitting your contribution, ensure the following: | ||
|
||
- [ ] Issue created and linked in the PR | ||
- [ ] Branch created from `main` and appropriately named | ||
- [ ] Tests written and passing | ||
- [ ] Documentation updated (if applicable) | ||
- [ ] Code formatted and linted | ||
- [ ] Changes thoroughly explained in the PR description |
Oops, something went wrong.