feat: Initial release #1
Conversation
action.yml
Outdated
| description: Response data | ||
|
|
||
| runs: | ||
| using: 'node12' |
action.yml
Outdated
|
|
||
| runs: | ||
| using: 'node12' | ||
| main: 'src/index.js' |
| setFailed('error status: ' + error.status); | ||
| } | ||
|
|
||
| run(); |
There was a problem hiding this comment.
Pull Request Overview
This PR introduces the initial release of a GitHub Action that updates GitHub variables for repositories and organizations. Key changes include implementation of the core logic in src/index.js, the action metadata in action.yml, and various configuration and workflow files to support CI/CD and release management.
Reviewed Changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/index.js | Implements the logic to update GitHub variables using Octokit and node-fetch. |
| package.json | Declares project metadata, dependencies, and scripts for building and linting. |
| action.yml | Defines the action’s inputs, outputs, and runtime environment. |
| SECURITY.md, README.md, LICENSE, etc. | Provide supporting documentation, licensing, and workflow configurations. |
|
|
||
| return octokit.request(`PATCH /${parameters.base}/${parameters.repository}/actions/variables/${parameters.name}`, { | ||
| value: parameters.value, | ||
| selected_repository_ids: parameters.selectedRepositoryIds, |
There was a problem hiding this comment.
The parameter 'selectedRepositoryIds' is obtained as a string from getInput but is passed directly to the API as selected_repository_ids, which likely expects an array. Consider parsing or converting this input to an array if necessary.
action.yml
Outdated
| description: Response data | ||
|
|
||
| runs: | ||
| using: 'node12' |
There was a problem hiding this comment.
The action specifies using 'node12' as the runtime although the code uses ES module syntax, which may not be fully supported by Node.js 12. Consider updating the runtime to a newer Node version (e.g. node16) to ensure compatibility.
| using: 'node12' | |
| using: 'node16' |
No description provided.