chore(cli): checlist for opensource#165
Merged
Merged
Conversation
CI Summary
Release - PassedTest this PR Download artifact (GitHub CLI required): gh run download 26992739723 -n cli-release-0.0.0-pr.165 -R paritytech/dotns-sdkInstall globally: npm install -g ./parity-dotns-cli-0.0.0-pr.165.tgzVerify: dotns --helpDeploy UI — FailedFailed at: Deploy — Deploy workflow failed — see run logs for upload/register/contenthash details
Deploy Example — Passed
Benchmark UI - Passed (169s)
Labelspkg: cli, type: docs, dependencies, pkg: ui, scope: benchmark Test - Passed229 passed, 0 failed across 229 tests. |
andrew-ifrita
approved these changes
Jun 5, 2026
andrew-ifrita
left a comment
Collaborator
There was a problem hiding this comment.
This looks fine but keep in mind that this repo is already public
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Prepares dotns-sdk for open sourcing. Declares the licence in three places (root
LICENSE, the licence field on the root and workspace packages, and a README note) and adds a LICENSE inside the publishable CLI package so it ships in the npm tarball. Adds SECURITY.md and CONTRIBUTING.md, a README status section flagging the code as reference-only and not audited, and secret scanning (a gitleaks config plus a workflow that runs on every pull request and push to main). Adds AI-assistant ignore rules to .gitignore. Also adds no-cache to every setup-bun step, so the action stops restoring a cache that lacks the bun binary.Type
Package
@parity/dotns-cliRelated Issues
None.
Fixes
setup-bun jobs failing with "Unable to locate executable file: ~/.bun/bin/bun" when a cache without the binary is restored.
Checklist
Code
bun run lintpassesbun run formatpassesbun run typecheckpassesDocumentation
Breaking Changes
Breaking changes:
None.
Testing
How to test:
gitleaks dir . --config .gitleaks.tomland a full-history scan (gitleaks git . --log-opts="--all" --config .gitleaks.toml). Both report no leaks.bun run format. It passes for the CLI, the UI, and the workflow YAML.LICENSE,SECURITY.md, andCONTRIBUTING.mdexist, and that the README has Status and License sections.Notes
The change set is documentation, CI, and configuration only; no TypeScript source changed, so lint and typecheck are unaffected, and format was run and passes. Dependency licences were audited: no copyleft reaches the published MIT CLI package; the GPL dependencies are confined to the private, unpublished UI app. ABI sync depends on the DotNS contracts repository being public with a release, which is handled separately.