Skip to content

Latest commit

 

History

History
220 lines (149 loc) · 12.5 KB

CONTRIBUTING.md

File metadata and controls

220 lines (149 loc) · 12.5 KB

Contributing to Mercury

🚀🎉 First off, thanks for taking the time to contribute! 🎉🚀
The following is a set of guidelines for contributing to Mercury. These are mostly guidelines. Use your best judgment, and feel free to propose changes to this document in a pull request.

What should I know before I get started?

Atala PRISM

Atala PRISM is a self-sovereign identity (SSI) platform and service suite for verifiable data and digital identity. Built on Cardano, it offers core infrastructure for issuing DIDs (Decentralized identifiers) and verifiable credentials, alongside tools and frameworks to help expand your ecosystem.
The complete platform is separated in multiple repositories:

  • Apollo - Repo for the Apollo Building Block, this contains the collection of the cryptographic methods used all around Atala PRISM.
  • atala-prism-building-blocks - Repo that contains the servers Building Blocks.
  • atala-prism-kmm-sdk - Repo that represents the Kotlin Multiplatform SDK.

What is Mercury

Kotlin Multiplatform with support for the following targets:

  • JS
  • iOS
  • Android
  • JVM

Future target might be supported in the future.

Your First Code Contribution

Unsure where to begin contributing to Apollo? You can start by looking through the Readme that provides all the steps to set up your environment.

Pull Requests

The process described here has several goals:

  • Maintain the SDK quality
  • Fix problems that are important to users
  • Engage the community in working toward the best possible product
  • Enable a sustainable system for the SDK maintainers to review contributions

Please follow these steps to have your contribution considered by the maintainers:

  1. Follow all instructions in the template
  2. Follow the Styleguide
  3. After you submit your pull request, verify that all status checks are passing
    What if the status checks are failing?If a status check is failing, and you believe that the failure is unrelated to your change, please leave a comment on the pull request explaining why you believe the failure is unrelated. A maintainer will re-run the status check for you. If we conclude that the failure was a false positive, then we will open an issue to track that problem with our status check suite.

While the prerequisites above must be satisfied prior to having your pull request reviewed, the reviewer(s) may ask you to complete additional design work, tests, or other changes before your pull request can be ultimately accepted.

Reporting Bugs

This section guides you through submitting a bug report for Apollo Atala PRISM. Following these guidelines helps maintainers and the community understand your report 📝, reproduce the behavior 💻 💻, and find related reports 🔎.

Before creating bug reports, please check this list as you might find out that you don't need to create one. When you are creating a bug report, please include as many details as possible. Fill out the required template, the information it asks for helps us resolve issues faster.

Note: If you find a Closed issue that seems like it is the same thing that you're experiencing, open a new issue and include a link to the original issue in the body of your new one.

  • You might be able to find the cause of the problem and fix things yourself by Debugging. Most importantly, check if you can reproduce the problem in the latest version.
  • Check the Readme ** if you have problems on the setup and the discussions for a list of common questions and problems.
  • Perform a cursory search to see if the problem has already been reported. If it has and the issue is still open, add a comment to the existing issue instead of opening a new one.

Bugs are tracked as GitHub issues. Create an issue on that repository and provide the following information by filling in the template.

Explain the problem and include additional details to help maintainers reproduce the problem:

  • Use a clear and descriptive title for the issue to identify the problem.
  • Describe the exact steps which reproduce the problem in as many details as possible.
  • Provide specific examples to demonstrate the steps. Include links to files or GitHub projects, or copy/paste able snippets, which you use in those examples. If you're providing snippets in the issue, use Markdown code blocks.
  • Describe the behavior you observed after following the steps and point out what exactly is the problem with that behavior.
  • Explain which behavior you expected to see instead and why.
  • If you're reporting that Apollo crashed, include a crash report with a stack trace from the operating system. On macOS, the crash report will be available in Console.app under "Diagnostic and usage information" > "User diagnostic reports". Include the crash report in the issue in a code block, a file attachment, or put it in a gist and provide link to that gist.
  • If the problem wasn't triggered by a specific action, describe what you were doing before the problem happened and share more information using the guidelines below.

Provide more context by answering these questions:

  • Did the problem start happening recently (e.g. after updating to a new version of Apollo) or was this always a problem?
  • If the problem started happening recently, can you reproduce the problem in an older version of Apollo? What's the most recent version in which the problem do not happen?
  • Can you reliably reproduce the issue? If not, provide details about how often the problem happens and under which conditions it normally happens.

Suggesting Enhancements

This section guides you through submitting an enhancement suggestion for the SDK, including completely new features and minor improvements to existing functionality. Following these guidelines helps maintainers and the community understand your suggestion 📝 and find related suggestions 🔎.

Before creating enhancement suggestions, please check this list as you might find out that you don't need to create one. When you are creating an enhancement suggestion, please include as many details as possible. Fill in the template, including the steps that you imagine you would take if the feature you're requesting existed.

  • Most importantly, check if you're using the latest version.
  • Perform a cursory search to see if the enhancement has already been suggested. If it has, add a comment to the existing issue instead of opening a new one.

Enhancement suggestions are tracked as GitHub issues. Create an issue on that repository and provide the following information:

  • Use a clear and descriptive title for the issue to identify the suggestion.
  • Provide a step-by-step description of the suggested enhancement in as many details as possible.
  • Provide specific examples to demonstrate the steps. Include copy/paste able snippets which you use in those examples, as Markdown code blocks.
  • Describe the current behavior and explain which behavior you expected to see instead and why.
  • Explain why this enhancement would be useful.
  • List some other text editors or applications where this enhancement exists.
  • Specify which version of Apollo you're using.
  • Specify the name and version of the OS you're using.

Styleguide

Git Commit Messages

Atala PRISM uses Conventional commits please always provide a commit following these specifications.

Commit Message Format

We have very precise rules over how our Git commit messages must be formatted.
This format leads to easier to read commit history.

Each commit message consists of a header, a body, and a footer.

<header>  
<BLANK LINE>  
<body>  
<BLANK LINE>  
<footer>  

The header is mandatory and must conform to the Commit Message Header format.

The body is mandatory for all commits except for those of type "docs".
When the body is present it must be at least 20 characters long and must conform to the Commit Message Body format.

The footer is optional. The Commit Message Footer format describes what the footer is used for and the structure it must have.

Commit Message Header

<type>(<scope>): <short summary>
  │       │             │
  │       │             └─⫸ Summary in present tense. Not capitalized. No period at the end.
  │       │
  │       └─⫸ Commit Scope: Hashing|DigitalSignatures|SymmetricEncryption|PublicKeyEncryption|Accumulators|MAC|KeExchange|Experience|Domain
  │
  └─⫸ Commit Type: build|ci|docs|feat|fix|perf|refactor|test

The <type> and <summary> fields are mandatory, the (<scope>) field is optional.

Type

Must be one of the following:

  • build: Changes that affect the build system or external dependencies
  • ci: Changes to the CI configuration files and scripts
  • docs: Documentation only changes
  • feat: A new feature
  • fix: A bug fix
  • perf: A code change that improves performance
  • refactor: A code change that neither fixes a bug nor adds a feature
  • test: Adding missing tests or correcting existing tests
Scope

The scope should be the name of the affected module or building block
(as perceived by the person reading the changelog generated from commit messages).

The following is the list of supported scopes:

  • Hashing
  • DigitalSignatures
  • SymmetricEncryption
  • PublicKeyEncryption
  • Accumulators
  • MAC
  • KeExchange
  • Experience
  • Domain
Summary

Use the summary field to provide a succinct description of the change:

  • use the imperative, present tense: "change" not "changed" nor "changes"
  • don't capitalize the first letter
  • no dot (.) at the end

Commit Message Body

Just as in the summary, use the imperative, present tense: "fix" not "fixed" nor "fixes".

Explain the motivation for the change in the commit message body. This commit message should explain why you are making the change.
You can include a comparison of the previous behavior with the new behavior in order to illustrate the impact of the change.

Commit Message Footer

The footer can contain information about breaking changes and deprecations and is also the place to reference GitHub issues, Jira tickets, and other PRs that this commit closes or is related to.
For example:

BREAKING CHANGE: <breaking change summary>  
<BLANK LINE>  
<breaking change description + migration instructions>  
<BLANK LINE>  
<BLANK LINE>  
Fixes ATL-<issue number>  

or

DEPRECATED: <what is deprecated>  
<BLANK LINE>  
<deprecation description + recommended update path>  
<BLANK LINE>  
<BLANK LINE>  
Related to ATL-<issue number>  

Breaking Change section should start with the phrase "BREAKING CHANGE: " followed by a summary of the breaking change, a blank line, and a detailed description of the breaking change that also includes migration instructions.

Similarly, a Deprecation section should start with "DEPRECATED: " followed by a short description of what is deprecated, a blank line, and a detailed description of the deprecation that also mentions the recommended update path.

Kotlin Styleguide

The Kotlin style guide is maintained by tools like ktlint. That will follow the Official Kotlin Style Guide.