Skip to content

Bump dependencies - #685

Open
KotRikD wants to merge 1 commit into
masterfrom
chore/bump_deps
Open

Bump dependencies#685
KotRikD wants to merge 1 commit into
masterfrom
chore/bump_deps

Conversation

@KotRikD

@KotRikD KotRikD commented Aug 5, 2026

Copy link
Copy Markdown
Member

PR made using AI tools

Claude Opus 5

@KotRikD KotRikD changed the title chore: auto-bump dependencies Bump dependencies Aug 5, 2026
@KotRikD KotRikD added the ai-written Not human made :( label Aug 5, 2026
@GabuTheDev

Copy link
Copy Markdown
Member

Did you really use Opus 5 to bump some dependencies????????????????????????????????????????? bruh

@cyperdark

Copy link
Copy Markdown
Collaborator

needed review from @storycraft, since there was a bump of electron dependencies which used by ingame overlay

@cyperdark
cyperdark requested a review from storycraft August 5, 2026 18:33
"@types/node": "^24.10.4",
"electron": "^39.2.7",
"electron-builder": "^26.4.0",
"electron": "^40.1.0",

@storycraft storycraft Aug 5, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Electron overlay renderer expect specific version of Electron(defined as peerDependencies). They can introduce breaking changes between major versions. In fact, some previous versions did before.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't test the build. But if it works without any issues, you can resolve this and ignore.

@storycraft

Copy link
Copy Markdown
Member

needed review from @storycraft, since there was a bump of electron dependencies which used by ingame overlay

Yes thank you for noticing this. Because it actually does matter.

@KotRikD

KotRikD commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Did you really use Opus 5 to bump some dependencies????????????????????????????????????????? bruh

¯\_(ツ)_/¯ claude sub, why not

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-written Not human made :(

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants