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 Update test after recent mods #1336

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Conversation

jarzec
Copy link
Contributor

@jarzec jarzec commented Nov 9, 2024

Updates results of pure2-last-use.cpp2 for a number of compilers, including clang-18.
The results for clang-18 were generated with the fix from #1335 applied.
Together with #1335 this PR makes all the tests pass on CI.

@jarzec jarzec force-pushed the ci-update-tests branch 2 times, most recently from e8a4c5a to e5ea718 Compare January 5, 2025 15:23
@jarzec
Copy link
Contributor Author

jarzec commented Jan 5, 2025

@hsutter I saw you have recently pushed to the repo, so I've updated this PR as well as #1335. As stated in the description, those two PRs make all test pass on the CI without functional changes. Hence, I wanted to ask you to send me the new CLA so that the two PR could be merged.

@hsutter
Copy link
Owner

hsutter commented Jan 15, 2025

Thanks! CLA sent for this and the other PR, thanks for waiting.

@jarzec
Copy link
Contributor Author

jarzec commented Jan 19, 2025

I've rebased the branch on current main.
The PR is ready to be merge (the remaining build failures are solved in #1335).

@hsutter
Copy link
Owner

hsutter commented Jan 31, 2025

Thanks!

@hsutter hsutter merged commit 8b36260 into hsutter:main Jan 31, 2025
26 of 29 checks passed
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