Skip to content

Add automated package content verification#34

Merged
blackopsrepl merged 2 commits intomainfrom
issue-15-package-verify
Mar 21, 2026
Merged

Add automated package content verification#34
blackopsrepl merged 2 commits intomainfrom
issue-15-package-verify

Conversation

@blackopsrepl
Copy link
Copy Markdown
Contributor

Closes #15.

Summary

  • add a repository-owned package verification script based on cargo package --list
  • fail CI and publish workflows if required packaged files are missing or excluded files leak into the crate
  • expose the check locally via make package-verify and include it in the pre-release path
  • document the verification workflow in the README

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ff3c45f5e1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +52 to +53
- name: Verify package contents
run: ./scripts/verify-package.sh
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Mirror package verification into Forgejo CI

Checked both CI definitions in this repo: .github/workflows/ci.yml now runs ./scripts/verify-package.sh, but .forgejo/workflows/ci.yml still stops after cargo test. If PRs or release branches are validated on the Forgejo runner, missing packaged assets or leaked excluded files will still pass there, so this automation only protects one of the maintained CI entry points.

Useful? React with 👍 / 👎.

@blackopsrepl blackopsrepl force-pushed the issue-15-package-verify branch from ff3c45f to 07bfefc Compare March 21, 2026 13:14
@blackopsrepl blackopsrepl merged commit ed59f1c into main Mar 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add reproducible package verification for the published crate contents

1 participant