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

add shell-based regression-tests #5512

Merged
merged 2 commits into from
Dec 19, 2024
Merged

add shell-based regression-tests #5512

merged 2 commits into from
Dec 19, 2024

Conversation

aryairani
Copy link
Contributor

@aryairani aryairani commented Dec 19, 2024

The issue in #5507 involves the compile command, and testing the resulting executable, which we can't currently do from a transcript alone. This PR adds a scripts section to CI and a script to test this.

this specific test may become ineffective if the code serialization algorithm someday includes termlinks as dependencies.
@aryairani aryairani requested a review from a team as a code owner December 19, 2024 17:20
@aryairani aryairani changed the title add regression test for #5507 add shell-based regression-tests Dec 19, 2024
@aryairani aryairani force-pushed the inlining-regression-test branch from 8c4e29a to 159c2b3 Compare December 19, 2024 17:29
@aryairani aryairani merged commit a2d3c2c into trunk Dec 19, 2024
32 checks passed
@aryairani aryairani deleted the inlining-regression-test branch December 19, 2024 19:17
@aryairani
Copy link
Contributor Author

cc @ceedubs

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