Skip to content

Latest commit

 

History

History
93 lines (57 loc) · 4.12 KB

contributing.md

File metadata and controls

93 lines (57 loc) · 4.12 KB
Raw

Contributing guidelines

Before you create a new PR on the Saga Security Chain repo, make sure that you read and comply with this document.

To prepare for success, see installing Saga Security Chain.

Thank you for your contribution!

Providing Feedback

  • Before you open an issue, do a web search, and check for existing open and closed GitHub Issues to see if your question has already been asked and answered. If you find a relevant topic, you can comment on that issue.

  • To provide feedback or ask a question, create a GitHub issue. Be sure to provide the relevant information, case study, or informative links as suggested by the Pull Request template.

Opening pull requests (PRs)

Review the issues and discussions before you open a PR.

Choose a good PR title

Avoid long names in your PR titles. Make sure your title has fewer than 60 characters.

Follow Conventional Commits guidelines and keywords to find the best title.

Use parentheses to identify the package or feature that you worked on. For example: feat(services/chain) , fix(scaffolding), docs(migration).

Review your own code

Make sure that you manually tested the changes you're introducing before creating a PR or pushing another commit.

Monitor your PR to make sure that all CI checks pass and the PR shows All checks have passed (the checkmark is green).

Do not rebase commits in your branch

Avoid rebasing after you open your PRs to reviews. Instead, add more commits to your PR. It's OK to do force pushes if the PR is still in draft mode and was never opened to reviews before.

A reviewer likes to see a linear commit history while reviewing. If you tend to force push from an older commit, a reviewer might lose track in your recent changes and will have to start reviewing from scratch.

Don't worry about adding too many commits. The commits are squashed into a single commit while merging. Your PR title is used as the commit message.

Contributing to documentation

When you open a PR for the SSC codebase, you must also update the relevant documentation. For changes to:

Ask for help

If you started a PR but couldn't finish it for whatever reason, don't give up. Instead, just ask for help. Someone else can take over and assume the ownership.

Prioritizing issues with milestones

Saga Security Chain follows Git Flow for branch strategy https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow.

  • Each SSC release has a milestone, see link.

  • Issues in each milestone have a priority/high, priority/medium, or priority/low label.

    • Select issues to work on for the earliest milestone. For example, select to work on an issue labeled as ** priority/low** in milestone v0.1.0 before you work on an issue labeled as priority/high in milestone v0.2.0.

  • Milestone Next is applied to issues that suggest adding features, docs, and so on.

    • Issues with the Next milestone have a higher priority than other Issues with no milestone (no milestone is assigned).

    • Issues in the Next milestone usually have a lower priority than milestones that are associated with a release version, like Milestone v0.1.0.

We appreciate your contribution!