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

Use GHA env file to set output in CI #362

Merged
merged 2 commits into from Oct 20, 2022
Merged

Use GHA env file to set output in CI #362

merged 2 commits into from Oct 20, 2022

Conversation

ktasper
Copy link
Contributor

@ktasper ktasper commented Oct 20, 2022

This should fix the issue with this

Relevent GH Issue

@ktasper
Copy link
Contributor Author

ktasper commented Oct 20, 2022

As seen in my commit here I am not sure how best to deal with this in the test suite as Now its being saved off to an ENV var.

@EnricoMi
Copy link
Owner

This has been done for the code in #360. In the field there may be action runners that still support only the set-output, so the action has to work with both. The mentioned suite tests that fallback.

I completely forgot about the CI, thanks for fixing this.

@EnricoMi EnricoMi changed the title Fixed set-output deprecation Use GHA env file to set output in CI Oct 20, 2022
@github-actions
Copy link

github-actions bot commented Oct 20, 2022

Test Results (reference)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

Test Results (Dockerfile)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Docker Image)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (setup-python)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Test File)

97 tests  ±0   80 ✔️ ±0   3m 25s ⏱️ ±0s
  1 suites ±0   17 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Test Files)

     67 files  ±0         6 errors  358 suites  ±0   1h 6m 6s ⏱️ ±0s
1 682 tests ±0  1 396 ✔️ ±0    71 💤 ±0  208 ±0  7 🔥 ±0 
1 929 runs  ±0  1 589 ✔️ ±0  106 💤 ±0  226 ±0  8 🔥 ±0 

For more details on these parsing errors, failures and errors, see this check.

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (macOS python 3.6)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Linux python 3.6)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (macOS python installed)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Linux python installed)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Linux 22.04 python installed)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (macOS 12 python installed)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@github-actions
Copy link

Test Results (Windows python installed)

       63 files  ±0         63 suites  ±0   31m 59s ⏱️ + 1m 37s
     374 tests ±0       374 ✔️ ±0      0 💤 ±0  0 ±0 
23 562 runs  ±0  23 058 ✔️ ±0  504 💤 ±0  0 ±0 

Results for commit 7c3244c. ± Comparison against base commit 713caf1.

@EnricoMi
Copy link
Owner

@EnricoMi EnricoMi merged commit 71fba75 into EnricoMi:master Oct 20, 2022
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.

None yet

2 participants