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

CI Fixes #811

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

CI Fixes #811

wants to merge 17 commits into from

Conversation

tomjn
Copy link

@tomjn tomjn commented Jul 25, 2024

This was originally just to upgrade Node from v16 to v20 to fix the action breaking but a lot of other stuff was broken too so I've updated a number of things.

Attempts to address #810 to resolve issues using this action

@tomjn
Copy link
Author

tomjn commented Jul 25, 2024

Note that newer versions of the packages that were updated are available but I upgraded to the first Node 20 compatible versions to avoid breakage

@tomjn tomjn changed the title Switch from Node v16 to Node v20 CI Fixes Jul 25, 2024
@svandragt
Copy link

svandragt commented Sep 10, 2024

If you brew install swiftlint when on macos SwiftLint will be found and the last test might pass. @tomjn
Source

Copy link
Contributor

Thank you for your contribution to this project!

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants