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

Update SV testfiles #29

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Update SV testfiles #29

merged 1 commit into from
Nov 9, 2023

Conversation

lorenzschmid
Copy link
Contributor

@lorenzschmid lorenzschmid commented Nov 6, 2023

The combination of the two merge requests #27 and #28 were not compatible. As the GitHub Action job runs only upon pushing, and the two merge request were shortly merged one after another, it was not detected. This PR updates the missing/outdated testfiles.

@stefanlippuner
Copy link
Contributor

I think the problem was triggered because the changes #27 and #28 were not compatible. Thanks for pushing a fix here!

It is unclear to me how the GitHub action jobs in #27 passed without
error. Also, I normally run all tests before pushing. Still, it seems
that some updated SV testfiles were likely be masked by a .gitignore
file and not committed in #27. Or, the combination of the two merge
requests #27 and #28 were not compatible.

This oversight should have been caught by the GitHub action job. As it
did not, here the remaining missing files.
@lorenzschmid
Copy link
Contributor Author

lorenzschmid commented Nov 6, 2023

@stefanlippuner yes, I thin you are right. I just arrived at the same conclusion and updated this PR's description accordingly.

@tgingold-cern tgingold-cern merged commit ef4bebc into tgingold-cern:master Nov 9, 2023
2 checks passed
@lorenzschmid lorenzschmid deleted the fix-pipeline-buffer branch November 9, 2023 11:28
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.

3 participants