Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 58 additions & 1 deletion contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,27 @@ Most files in the project are autogenerated by [openapi-generator](https://opena
- [docs](./docs) - generated documentation for models and [API Client](docs/Api/FingerprintApi.md).
- [src](./src) - API Client code and generated models.

## Commit messages

This project follows the [Conventional Commits](https://www.conventionalcommits.org/) standard. [commitlint](https://commitlint.js.org/) checks the messages of all commits in a pull request in the `Analyze Commit Messages` check, using the [@fingerprintjs/commit-lint-dx-team](https://www.npmjs.com/package/@fingerprintjs/commit-lint-dx-team/v/0.1.0) config.

## Git hooks

This repository includes optional Git hooks in the [.git_hooks](./.git_hooks) folder. To enable them, run:

```shell
pnpm install
./install_hooks.sh
```

The script sets `core.hooksPath` to `.git_hooks` and installs commitlint globally with npm, so you need Node.js. The hooks do the following:

- `commit-msg` checks the commit message with commitlint and rejects the commit if the message is invalid.
Comment thread
erayaydin marked this conversation as resolved.
- `pre-commit` runs `./scripts/php-cs-fixer.sh` to fix the code style.
- `pre-push` tries to stop accidental pushes to `main`.

Commit messages are also checked in CI. If the check fails, reword the offending commits (for example, with `git rebase -i`) and force-push the branch.

## Code generation

You need `openapi-generator` to run code generation. There are many ways described in the [readme](https://github.com/OpenAPITools/openapi-generator).
Expand Down Expand Up @@ -76,4 +97,40 @@ docker run --rm openapitools/openapi-generator-cli:v7.23.0 config-help -g php

### How to publish

We use [changesets](https://github.com/changesets/changesets) for handling release notes. If there are relevant changes, please add them to changeset via `pnpm exec changeset`. You need to run `pnpm install` before doing so.
We use [changesets](https://github.com/changesets/changesets) to version the SDK and to write release notes.

#### Adding a changeset

If your PR changes the SDK's public API or behavior, add a changeset to it:

```shell
pnpm install
pnpm exec changeset
```

Pick the bump type and write a short summary. The command creates a markdown file in the [.changeset](./.changeset) folder. Commit it together with the rest of your changes. The summary is copied as-is into `CHANGELOG.md` and the GitHub release notes, so write it for SDK users:

```md
---
'@fingerprint/php-sdk': minor
---

Add `device_details` smart signal to the event model
```

Pick the bump type that matches the commit type:

| Change | Commit type | Changeset bump | Version change |
|---|---|---|---|
| Bug fix | `fix` | `patch` | 7.7.0 -> 7.7.1 |
| New backward-compatible feature | `feat` | `minor` | 7.7.0 -> 7.8.0 |
| Breaking change | Any `<type>!` (for example, `feat!`) or a `BREAKING CHANGE:` footer | `major` | 7.7.0 -> 8.0.0 |
| Docs, tests, CI, refactoring and other internal changes | `docs`, `test`, `ci`, `refactor`, `chore`, ... | No changeset | No release |

If a PR has several user-facing changes, add one changeset for each. When several changesets are released together, the highest bump wins.

#### Release flow

1. On every PR, a bot comments with a preview of the release notes that the PR's changesets will produce. If the PR has no changesets, the comment reminds you to add one.
2. After a PR with changesets is merged to `main`, the [Release](./.github/workflows/release.yml) workflow opens a `Release [changeset]` PR, or updates it if it's already open. That PR consumes all pending changesets, bumps the version and updates `CHANGELOG.md`.
3. Merging the `Release [changeset]` PR creates the Git tag and the GitHub release. [Packagist](https://packagist.org/packages/fingerprint/server-sdk) picks up the new tag.
Loading