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

^ Update os references in action configs #51

Merged
merged 1 commit into from
Dec 15, 2024
Merged

Conversation

PikachuEXE
Copy link
Owner

Ubuntu latest will soon be switched to 24 LTS

@PikachuEXE PikachuEXE force-pushed the update/action-os branch 2 times, most recently from a1d486c to 800196c Compare December 13, 2024 03:43
@PikachuEXE
Copy link
Owner Author

@toobuntu See if you want to take a look

@toobuntu
Copy link
Contributor

Some general thoughts:

  • Glad to see this. I had been working on a similar PR but haven't had time to finish it.
  • With macOS Sequoia, Apple removed the Ctrl-click method to override Gatekeeper. As I understand it, if software cannot be run on macOS, Homebrew will eventually deprecate and then disable and remove it from its ecosystem.[1,2] So FreeTube might want to reconsider hosting an official tap for both Intel and Apple Silicon casks, with the --no-quarantine instruction. Or you could add Intel to this tap.
  • Now that macos-latest points to macos-14, I would say reduce the future maintenance burden by using macos-latest and ubuntu-latest everywhere.
  • Although the Homebrew Docker image is on hub, the Homebrew project leader prefers to use ghcr.io: . You can find ubuntu24.04 here: https://github.com/orgs/Homebrew/packages?repo_name=brew.
  • In general, the Actionlint part of things in this tap's CI has become a bit of a mess. The ETags still update but not the content anymore, and upstream is now uploading CodeQL CLI SARIF output. In other words, I think it's time to diverge from upstream and stop trying to sync with its changes. We can take a look at the Actionlint workflow and make it good enough for this tap and leave it at that.

[1] Homebrew/brew#16752
[2] https://github.com/Homebrew/brew/blob/master/Library/Homebrew/deprecate_disable.rb#L28-L30

@PikachuEXE
Copy link
Owner Author

@toobuntu
Copy link
Contributor

I see you tried

  • ghcr.io/homebrew/ubuntu24.04:master, which does not exist for 24.04. So I would expect that to fail.
  • ghcr.io/homebrew/ubuntu24.04:cache, which does exist. I don't know why it failed.

@toobuntu
Copy link
Contributor

toobuntu commented Dec 15, 2024

Have you tried ghcr.io/homebrew/ubuntu24.04:latest?
It worked over here: https://github.com/jck112/homebrew-cask/actions/runs/12326368226/job/34407056594.

# Use macos-14 for arm64 architecture: https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/
# macos-latest refers to the macos-12 (x86_64) runner image until April–June 2024: https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/ and https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories
runs-on: macos-14
# macos-latest refers to macOS 14 Arm64
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# macos-latest refers to macOS 14 Arm64
# macos-latest refers to macOS on arm64

To avoid having to update this when latest refers to 15+, might as well just reference the important part which is that we need arm and latest is now arm instead of Intel.

Ubuntu latest will soon be switched to 24 LTS
@PikachuEXE
Copy link
Owner Author

Updated

@toobuntu
Copy link
Contributor

LGTM

@PikachuEXE PikachuEXE merged commit 375a2be into master Dec 15, 2024
4 checks passed
@PikachuEXE PikachuEXE deleted the update/action-os branch December 15, 2024 02:53
@PikachuEXE
Copy link
Owner Author

Should we just remove .github/workflows/update-actionlint-workflow.yml since there is really no need to update it to be like upstream? (besides it keep creating PRs about etag changes

@toobuntu
Copy link
Contributor

Yes, and there is no need to keep the etags dir around, either.

rm -rf .github/workflows/update-actionlint-workflow.yml .github/etags

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