Skip to content

Implement SDK pre-submission verification command - #422

Merged
El-swaggerito merged 1 commit into
Axionvera:mainfrom
panditdhamdhere:feat/presubmit-verification-command-392
Jul 28, 2026
Merged

Implement SDK pre-submission verification command#422
El-swaggerito merged 1 commit into
Axionvera:mainfrom
panditdhamdhere:feat/presubmit-verification-command-392

Conversation

@panditdhamdhere

Copy link
Copy Markdown
Contributor

Summary

  • Add npm run presubmit (verify:presubmit) — one local command that runs lint, circular-deps, unit tests, coverage, and build before opening a PR
  • Add scripts/presubmit.ts with labeled steps and per-step failure hints
  • Document usage and failure guidance in docs/pre-submission-verification.md
  • Wire README, CONTRIBUTING, testing docs, and the PR template to the command

Closes #392

Acceptance criteria

  • Verification command is added
  • Command runs relevant local checks
  • Documentation explains usage
  • Failure guidance is included
  • README links to command
  • Command is suitable for contributors before PR submission

Test plan

  • npm run presubmit -- --help prints usage
  • npm run presubmit runs the same CI-parity pipeline as npm run verify
  • Docs describe usage and failure guidance
  • Branch is able to merge into main

Introduce npm run presubmit to run the CI-parity lint/test/coverage/build
pipeline with step output and failure guidance before opening a PR.
@El-swaggerito
El-swaggerito merged commit efc840a into Axionvera:main Jul 28, 2026
1 check passed
@panditdhamdhere
panditdhamdhere deleted the feat/presubmit-verification-command-392 branch July 28, 2026 20:27
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.

Implement SDK pre-submission verification command

2 participants