Skip to content
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

Update @octokit/rest to mitigate transitive CVEs #6

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

fbartho
Copy link
Contributor

@fbartho fbartho commented Mar 11, 2025

Reference:

I updated package.json dependencies, and reinstalled modules, deleted the build directory and then ran yarn build and there were no build errors, and no changes in the build/ directory.

For good measure, I also opened index.ts, and VSCode didn't give me any warnings or TS Errors.

Reference:
- https://github.com/octokit/rest.js/releases/tag/v21.1.1
- danger/danger-js#1477
- danger/danger-js#1479

I updated package.json dependencies, and reinstalled modules, deleted the `build` directory and then ran `yarn build` and there were no build errors, and no changes in the `build/` directory.

For good measure, I also opened `index.ts`, and VSCode didn't give me any warnings or TS Errors.
@orta
Copy link
Owner

orta commented Mar 12, 2025

I'm not gonna be wasting my time making releases for specific versions of octokit, so I'm gonna switch it to a "*'

@orta orta merged commit 6e24ce8 into orta:main Mar 12, 2025
@orta
Copy link
Owner

orta commented Mar 12, 2025

Deployed as 1.3.0

@fbartho fbartho deleted the fb/update-octokit-rest branch March 12, 2025 16:11
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.

2 participants