Skip to content

Reduce Homebrew/brew CI warnings #20054

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 9, 2025
Merged

Reduce Homebrew/brew CI warnings #20054

merged 1 commit into from
Jun 9, 2025

Conversation

MikeMcQuaid
Copy link
Member

@MikeMcQuaid MikeMcQuaid commented Jun 5, 2025 •
edited
Loading

  • Remove a bunch of non-actionable/unnecessary noise in GitHub Actions CI.
  • Limit number of threads used to generate analytics API data to avoid reproducible failures producing errors and requiring retries.
  • Move to Debian Old Stable for testing non-system glibc.
  • Remove unneeded core taps/updates.
  • Improve naming of CI jobs to clarify purpose i.e. we're testing things work on Linux, not Ubuntu specifically.
  • Remove dedicated non-online/non-generic Linux brew tests jobs from 3 to 1.

@MikeMcQuaid MikeMcQuaid force-pushed the remove_ci_warnings branch from c83f272 to 062ec97 Compare June 5, 2025 15:37
Rylan12
Rylan12 approved these changes Jun 5, 2025
View reviewed changes
Copy link
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

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

Nice!

One comment but otherwise looks great

@MikeMcQuaid MikeMcQuaid force-pushed the remove_ci_warnings branch 6 times, most recently from 7960666 to f215c52 Compare June 6, 2025 07:38
github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 6, 2025
View reviewed changes
@MikeMcQuaid MikeMcQuaid force-pushed the remove_ci_warnings branch 21 times, most recently from a58fef8 to 0c550b8 Compare June 9, 2025 07:53
@MikeMcQuaid MikeMcQuaid force-pushed the remove_ci_warnings branch 4 times, most recently from 3951dcb to cadb6ee Compare June 9, 2025 08:25
@MikeMcQuaid @Rylan12
Reduce Homebrew/brew CI warnings
cc03340 
- Remove a bunch of non-actionable/unnecessary noise in GitHub Actions
  CI.
- Limit number of threads used to generate analytics API data to avoid
  reproducible failures producing errors and requiring retries.
- Move to Debian Old Stable for testing non-system `glibc`.
- Remove unneeded core taps/updates.
- Improve naming of CI jobs to clarify purpose i.e. we're testing
  things work on Linux, not Ubuntu specifically.
- Remove dedicated non-online/non-generic Linux `brew tests` jobs from
  3 to 1.

Co-authored-by: Rylan Polster <rslpolster@gmail.com>
@MikeMcQuaid MikeMcQuaid force-pushed the remove_ci_warnings branch from cadb6ee to cc03340 Compare June 9, 2025 11:47
@MikeMcQuaid MikeMcQuaid added this pull request to the merge queue Jun 9, 2025
Merged via the queue into master with commit 97bc7b1 Jun 9, 2025
36 checks passed
@MikeMcQuaid MikeMcQuaid deleted the remove_ci_warnings branch June 9, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rylan12 Rylan12 Rylan12 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeMcQuaid @Rylan12