-
Notifications
You must be signed in to change notification settings - Fork 0
release: 0.0.2 #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
release: 0.0.2 #1
Conversation
|
🧪 Testing To try out this version of the SDK, run: Expires at: Wed, 21 Jan 2026 18:21:16 GMT |
|
|
||
| lenErrors=${#errors[@]} | ||
|
|
||
| if [[ lenErrors -gt 0 ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing $ prefix causes error check to always fail
The condition uses lenErrors instead of $lenErrors. In bash, without the $ prefix, lenErrors is treated as a string literal rather than a variable reference. This means the check will never correctly detect errors - even if the errors array were populated, the condition would fail to trigger the error handling block, and the script would always report "The environment is ready to push releases!" regardless of actual errors.
| release_doctor: | ||
| name: release doctor | ||
| runs-on: ubuntu-latest | ||
| if: github.repository == 'onkernel/hypeman-ts' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch' || startsWith(github.head_ref, 'release-please') || github.head_ref == 'next') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Workflow condition checks for impossible push event
The if condition checks for github.event_name == 'push', but the workflow only triggers on pull_request and workflow_dispatch events. The push condition will never be true. This appears to be a copy-paste error - the condition likely should check for pull_request instead of push to correctly match the workflow triggers.
bee8267 to
d39b362
Compare
|
🤖 Release is at https://github.com/onkernel/hypeman-ts/releases/tag/v0.0.2 🌻 |
Automated Release PR
0.0.2 (2025-12-22)
Full Changelog: v0.0.1...v0.0.2
Chores
This pull request is managed by Stainless's GitHub App.
The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.
🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions
Note
Release and package scope update
@onkernel/hypemanand updates all imports, path mappings, Jest/Eslint configs, build scripts, and tests accordingly0.0.2(src/version.ts,.release-please-manifest.json) and addsCHANGELOG.mdrelease-please-config.json, GitHub workflowsPublish NPMandRelease Doctor, plusbin/check-release-environmentREADME.md,CONTRIBUTING.md, repository metadata (package.jsonrepo/name), and workspace config.stats.ymlconfig hash)Written by Cursor Bugbot for commit d39b362. This will update automatically on new commits. Configure here.