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

github: check: print logs in case of errors #320

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

matttbe
Copy link
Member

@matttbe matttbe commented Feb 10, 2025

Otherwise, only a vague error is printed:

 ../test-driver: line 112: 18416 Aborted (core dumped) "$@" >> "$log_file" 2>&1

@coveralls
Copy link

coveralls commented Feb 10, 2025

Pull Request Test Coverage Report for Build 13239188394

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 64.104%

Totals Coverage Status
Change from base Build 13237932222: 0.0%
Covered Lines: 1409
Relevant Lines: 2198

💛 - Coveralls

@matttbe matttbe force-pushed the gh-check-error branch 3 times, most recently from 84905ac to 7ed4542 Compare February 10, 2025 10:42
Otherwise, only a vague error is printed:

   ../test-driver: line 112: 18416 Aborted (core dumped) "$@" >> "$log_file" 2>&1

Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
@matttbe matttbe force-pushed the gh-check-error branch 2 times, most recently from 00acc9f to 5241da9 Compare February 10, 2025 10:53
@matttbe matttbe merged commit e269bbc into multipath-tcp:main Feb 10, 2025
6 of 8 checks passed
@matttbe matttbe deleted the gh-check-error branch February 10, 2025 10:54
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