Skip to content

Conversation

@rafaeelaudibert
Copy link
Contributor

What has changed?

This update introduces improved exit codes for the sampo commands, allowing for easier detection of no-op scenarios. The changes include modifications to the command implementations to return specific exit codes, enhancing usability in CI/CD pipelines. Additionally, the README has been updated to document the new exit codes and their meanings.

How is it tested?

Existing tests still pass. Not sure if it makes sense to have tests for these?

How is it documented?

Updated sampo README

This update introduces improved exit codes for the `sampo` commands, allowing for easier detection of no-op scenarios. The changes include modifications to the command implementations to return specific exit codes, enhancing usability in CI/CD pipelines. Additionally, the README has been updated to document the new exit codes and their meanings.
@sampo-s-bot
Copy link

sampo-s-bot bot commented Jan 19, 2026

🧭 Changeset detected

Merging this PR will release the following updates:

sampo (Cargo) — minor version bump

Minor changes

  • Improve exit codes to easily detect when running a sampo command would not apply any changes. This is useful in CI jobs when we don't want to run the whole process if there's no changes to be applied.

@github-actions github-actions bot added documentation Improvements or additions to documentation crt:sampo Changes in Sampo CLI crate. labels Jan 19, 2026
rafaeelaudibert added a commit to PostHog/posthog-elixir that referenced this pull request Jan 19, 2026
Sampo follows a process very similar to the `changesets` library we have in posthog-js and we love that behavior.

Let's implement it here in `elixir` too.

We depend on a couple fixes to `sampo` which I've submitted upstream: bruits/sampo#182 and bruits/sampo#183
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crt:sampo Changes in Sampo CLI crate. documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant