Skip to content

Conversation

@lukebakken
Copy link
Contributor

This PR uses the actions-up tool to update all GitHub actions to their latest release version, and uses commit SHAs to ensure only that version is used.

It also re-instates the Windows tests to see if they'll succeed.

@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.70%. Comparing base (2eed375) to head (852a2ff).
⚠️ Report is 119 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #447      +/-   ##
==========================================
- Coverage   84.50%   83.70%   -0.80%     
==========================================
  Files          26       28       +2     
  Lines        3756     4186     +430     
==========================================
+ Hits         3174     3504     +330     
- Misses        411      460      +49     
- Partials      171      222      +51     
Flag Coverage Δ
unittests 83.70% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lukebakken lukebakken force-pushed the lukebakken/github-actions-updates branch from 511d44e to 99baacf Compare November 24, 2025 19:06
@lukebakken
Copy link
Contributor Author

@Gsantomaggio all set for review. It looks like we've got at least one flaky test.

This PR uses the [`actions-up`](https://github.com/azat-io/actions-up)
tool to update all GitHub actions to their latest release version, and
uses commit SHAs to ensure only that version is used.

It also re-instates the Windows tests to see if they'll succeed.

* Fix Windows on GHA to not use make
* Build docker image as tar in separate step
* Use `golangci/golangci-lint` action in a separate workflow
* Move publish of go stream perf test docker image to its own workflow
* Use `nproc` to set `NUM_PROCS`
@lukebakken lukebakken force-pushed the lukebakken/github-actions-updates branch from 99baacf to 852a2ff Compare November 24, 2025 19:14
@Gsantomaggio
Copy link
Member

@lukebakken, thank you. Yes, there are some flaky tests. will check

@Gsantomaggio Gsantomaggio added this to the 1.6.2 milestone Nov 24, 2025
@Gsantomaggio Gsantomaggio self-assigned this Nov 24, 2025
@Gsantomaggio Gsantomaggio added the enhancement New feature or request label Nov 24, 2025
@Gsantomaggio Gsantomaggio merged commit fe10e46 into rabbitmq:main Nov 24, 2025
9 of 11 checks passed
@lukebakken lukebakken deleted the lukebakken/github-actions-updates branch November 24, 2025 19:57
@Gsantomaggio
Copy link
Member

Let's merge, then I will control the tests. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants