| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
I dont understand why #508 was a breaking change if it didnt remove any functionality and didnt change existing, it only added to the api surface. Can someone elaborate? |
Sorry, something went wrong.
If users are defining a variable named octokit in their scripts (which I suspect some are), then these scripts would fail with the changes introduced by #508 For example: - uses: actions/github-script@main
with:
script: |
const octokit = github
await octokit.rest.issues.createComment({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: ${{ github.event.issue.number }},
body: "Hello Again!"
})That will fail with the error Unhandled error: SyntaxError: Identifier 'octokit' has already been declared https://github.com/joshmgross/actions-testing/actions/runs/13551951964 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
#508 introduced a new octokit variable to make it easier to use examples from GitHub and Octokit docs within actions/github-script.
That change is a breaking change so the README updates shouldn't have been made with v7.
We'll update the README to include this info again when we're ready for v8 - #556