Skip to content

Fix workflows#149

Merged
Frostbyte0x70 merged 3 commits into
SkyTemple:masterfrom
Chesyon:master
Sep 29, 2025
Merged

Fix workflows#149
Frostbyte0x70 merged 3 commits into
SkyTemple:masterfrom
Chesyon:master

Conversation

@Chesyon

@Chesyon Chesyon commented Sep 27, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Doesn't really matter, I just need to make the actions run.
Switched from windows-2019 to windows-2022, since the former is deprecated and can no longer be used. (Attempted to) install dependencies for linting/checking.
Comment on lines +12 to +13
- name: Install Clippy
run: rustup component add clippy

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.

Are we sure this and the below addition are needed? Sounds kinda weird that it's only necessary to install them now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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.

Oh, well, I guess we can add it then.

@Frostbyte0x70 Frostbyte0x70 merged commit 2a15203 into SkyTemple:master Sep 29, 2025
11 checks passed
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