Skip to content

CI Update test after recent mods - #1336

Merged
hsutter merged 1 commit into
hsutter:mainfrom
jarzec:ci-update-tests
Jan 31, 2025
Merged

CI Update test after recent mods#1336
hsutter merged 1 commit into
hsutter:mainfrom
jarzec:ci-update-tests

Conversation

@jarzec

@jarzec jarzec commented Nov 9, 2024

Copy link
Copy Markdown
Contributor

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

jarzec commented Jan 5, 2025

Copy link
Copy Markdown
Contributor Author

@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

hsutter commented Jan 15, 2025

Copy link
Copy Markdown
Owner

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

@jarzec

jarzec commented Jan 19, 2025

Copy link
Copy Markdown
Contributor Author

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

@hsutter

hsutter commented Jan 31, 2025

Copy link
Copy Markdown
Owner

Thanks!

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