Release/0.1.1 - #12
Conversation
…unlisted/re-sign + new PAT screenshot
…token steps, screenshot, naming) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
chore: move CODEOWNERS to repo root (@BrkYld)
fix: axios version bump
📝 WalkthroughWalkthroughChangesEnterprise App Store product update
Repository ownership
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 75: Regenerate the checked-in action bundle after updating the axios
dependency in package.json, ensuring dist/index.js includes the upgraded Axios
version used by action.yml. Verify the generated bundle contains the intended
dependency version and commit the resulting dist/index.js changes.
In `@README.md`:
- Line 151: Resolve the mismatch between the README’s self-hosted Appcircle
Enterprise App Store documentation and the linked documentation page by updating
the README link or its surrounding hosting claim. Ensure the resulting guidance
consistently reflects the action’s actual Cloud and self-hosted support before
release.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: a6d896c0-b15c-4127-a985-78996fce6b31
⛔ Files ignored due to path filters (6)
dist/index.jsis excluded by!**/dist/**dist/index.js.mapis excluded by!**/dist/**,!**/*.mapdist/licenses.txtis excluded by!**/dist/**images/PAT.pngis excluded by!**/*.pngimages/ent_app_store.pngis excluded by!**/*.pngyarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (4)
CODEOWNERSREADME.mdaction.ymlpackage.json
| "dependencies": { | ||
| "@actions/core": "^1.10.1", | ||
| "axios": "^1.7.4", | ||
| "axios": "^1.18.1", |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Regenerate and commit the action bundle for the Axios upgrade.
action.yml runs dist/index.js, so changing package.json alone does not update the Axios version used by consumers. Rebuild the checked-in bundle and verify that dist/index.js contains the intended dependency version before release.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@package.json` at line 75, Regenerate the checked-in action bundle after
updating the axios dependency in package.json, ensuring dist/index.js includes
the upgraded Axios version used by action.yml. Verify the generated bundle
contains the intended dependency version and commit the resulting dist/index.js
changes.
|
|
||
| For more detailed instructions and support, visit the | ||
| [Appcircle Enterprise App Store documentation](https://appcircle.io/enterprise-app-store?utm_source=github&utm_medium=plugin&utm_campaign=enterprise_app_store). | ||
| [Appcircle Enterprise App Store documentation](https://docs.appcircle.io/marketplace/github-marketplace/enterprise-app-store?utm_source=github&utm_medium=plugin&utm_campaign=enterprise_app_store). |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Fix the documentation link/hosting claim mismatch.
The new target currently states that the action is compatible only with Appcircle Cloud and that self-hosted support is planned, while this README documents self-hosted support and configuration. Update one side before release so users are not sent to contradictory setup guidance. (docs.appcircle.io)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.md` at line 151, Resolve the mismatch between the README’s self-hosted
Appcircle Enterprise App Store documentation and the linked documentation page
by updating the README link or its surrounding hosting claim. Ensure the
resulting guidance consistently reflects the action’s actual Cloud and
self-hosted support before release.
Summary by CodeRabbit
Documentation
Chores