Skip to content

ci: Simplify our Install Dependencies action - #8608

Merged
nbolton merged 3 commits into
masterfrom
rmNinjainstallAction
May 20, 2025
Merged

ci: Simplify our Install Dependencies action#8608
nbolton merged 3 commits into
masterfrom
rmNinjainstallAction

Conversation

@sithlord48

@sithlord48 sithlord48 commented May 20, 2025

Copy link
Copy Markdown
Member

Simplifies the install dependencies action a bit:

  • remove the ninja install action as ninja is part of the runners now
  • remove cmake / ninja install from mac os runs as they are part of the runner for mac os also
  • Do do the Install Depends step on windows its was empty anyway

@sithlord48 sithlord48 added the 🤖 ci The item is for our CI label May 20, 2025
@sithlord48
sithlord48 requested a review from nbolton May 20, 2025 12:25
@sithlord48
sithlord48 force-pushed the rmNinjainstallAction branch from 486ad63 to abcd7d3 Compare May 20, 2025 12:56
@sithlord48 sithlord48 changed the title ci: remove install ninja action ninja is included on the runner now ci: remove install of item included on the runners May 20, 2025
@sithlord48
sithlord48 force-pushed the rmNinjainstallAction branch from abcd7d3 to 460c71f Compare May 20, 2025 13:00
@sithlord48 sithlord48 changed the title ci: remove install of item included on the runners ci: remove install of items included on the runners May 20, 2025
…ction on windows since it does nothing anyway
@sithlord48 sithlord48 changed the title ci: remove install of items included on the runners ci: Simplify our Install Dependencies action May 20, 2025
@nbolton
nbolton merged commit 12fbc2f into master May 20, 2025
@nbolton
nbolton deleted the rmNinjainstallAction branch May 20, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 ci The item is for our CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants