Skip to content

chore: fixed the husky setup#328

Open
denilsonsa wants to merge 8 commits into
Tormak9970:devfrom
denilsonsa:husky-fixes
Open

chore: fixed the husky setup#328
denilsonsa wants to merge 8 commits into
Tormak9970:devfrom
denilsonsa:husky-fixes

Conversation

@denilsonsa

Copy link
Copy Markdown

It is now automatically executed after npm install or npm ci or pnpm install --frozen-lockfile
https://docs.npmjs.com/cli/v11/using-npm/scripts

Also done chmod +x on the husky script, to prevent this error on unix-like systems:

hint: The '.husky/commit-msg' hook was ignored because it's not set as executable.
hint: You can disable this warning with git config set advice.ignoredHook false.

Also upgraded husky from v8 to v9.
https://github.com/typicode/husky/releases/tag/v9.0.1

I had to also upgrade some rollup-related dependencies to solve conflicts.

Tormak9970 and others added 8 commits January 11, 2026 12:43
Added installation instructions for Decky Store and ZIP method.
It is now automatically executed after `npm install` or `npm ci` or
`pnpm install --frozen-lockfile`
<https://docs.npmjs.com/cli/v11/using-npm/scripts>

Also done `chmod +x` on the husky script, to prevent this error on
unix-like systems:

> hint: The '.husky/commit-msg' hook was ignored because it's not set as executable.
> hint: You can disable this warning with `git config set advice.ignoredHook false`.

Also upgraded husky from v8 to v9.
<https://github.com/typicode/husky/releases/tag/v9.0.1>

I had to also upgrade some rollup-related dependencies to solve
conflicts.
@denilsonsa denilsonsa mentioned this pull request Jun 16, 2026
@denilsonsa

Copy link
Copy Markdown
Author

As a semi-related step… It might be a good idea to look at pnpm outdated and upgrade many of those dependencies.

For instance, I know uuid had some vulnerabilities in older versions, but those are not exploitable on the v4 module you're using. There were no breaking changes on v4, so uuid can be easily upgraded to the latest version without any issue.

@Tormak9970

Copy link
Copy Markdown
Owner

Looks good, but please target the Dev branch with your PR

@denilsonsa
denilsonsa changed the base branch from main to dev June 17, 2026 16:28
@denilsonsa

Copy link
Copy Markdown
Author

Changed to dev in the UI, but I think I need to rebase my branch. Gonna try that later. Alternatively, I noticed there are 7 commits in main that are not yet merged into dev. Shouldn't those be in dev already?

Also, I need to figure out how to test these in-development things on real SteamOS or at least the Big Picture Mode from a normal Linux installation.

@Tormak9970

Copy link
Copy Markdown
Owner

I'll sync up dev with main. For testing, big-picture on linux will work the same as on steamdeck. We don't use anything that's steamdeck specific. You'll just need to install decky

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