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

ci: update go.yml workflow to check output results for eligible demos #488

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aofei
Copy link
Member

@aofei aofei commented Jul 10, 2024

  • Modified the go.yml workflow to include output result checks for demos in _demo and _pydemo dirs.
  • Added conditions to compare the actual output with the expected output if the expected_output.txt file exists.
  • Enhanced the CI script to display detailed diff results when discrepancies are found, aiding in debugging and issue resolution.
  • Ensured that demos without expected_output.txt files still pass the CI checks.

- Modified the go.yml workflow to include output result checks for demos
  in _demo and _pydemo dirs.
- Added conditions to compare the actual output with the expected output
  if the expected_output.txt file exists.
- Enhanced the CI script to display detailed diff results when
  discrepancies are found, aiding in debugging and issue resolution.
- Ensured that demos without expected_output.txt files still pass the CI
  checks.
Copy link

codecov bot commented Jul 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.53%. Comparing base (d6a38a5) to head (a6f7662).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #488   +/-   ##
=======================================
  Coverage   97.53%   97.53%           
=======================================
  Files          19       19           
  Lines        4454     4454           
=======================================
  Hits         4344     4344           
  Misses         94       94           
  Partials       16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

1 participant