Skip to content

Update regression tests after recent changes #1208

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

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

bluetarpmedia
Copy link
Contributor

No description provided.

@hsutter hsutter merged commit df3f4ff into hsutter:main Aug 5, 2024
6 checks passed
@hsutter
Copy link
Owner

hsutter commented Aug 5, 2024

I hand-resolved that MSVC regex-10.execution file, it might not be quite right but for some reason it was sticking in my local GitHub environment's throat (it appeared to be a change I couldn't discard), so I just committed and pushed it and hand-resolved the conflict in this merge.

@bluetarpmedia bluetarpmedia deleted the update-regression-tests branch August 7, 2024 01:15
@MaxSagebaum
Copy link
Contributor

@bluetarpmedia Since you always do the regression updates. I just wanted to ask if there is any easy way to get all the diffs. Checking every version, searching for diff and then copying it out, is quite cumbersome. Is there an automated way to get all the diffs of all runners?

@bluetarpmedia
Copy link
Contributor Author

Yes, @jarzec modified the CI pipeline to create a handy aggregate of all the failed diffs.

Eg scroll down on this Summary page:
https://github.com/hsutter/cppfront/actions/runs/10327301935

Download the aggregate zip file, unzip into your cppfront directory and then git apply each diff file.

@DyXel
Copy link
Contributor

DyXel commented Aug 10, 2024

You probably can automate further with the Github CLI tool and/or a combination of scripts that uses Github APIs, probably something to look for in the future if more toolchains/targets are added.

@MaxSagebaum
Copy link
Contributor

Ah thank. That helped alot.

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.

4 participants