Skip to content

Commit

Permalink
- fix lint prolbems
Browse files Browse the repository at this point in the history
  • Loading branch information
mikepenz committed Aug 20, 2022
1 parent 191ca70 commit 3fdd534
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 5 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,15 @@ jobs:
report_paths: '**/surefire-reports/TEST-*.xml'
summary: '<table><thead><tr><th> Application (src/applications) </th></tr></thead><tbody><tr><td> test </td></tr></tbody></table>'
check_title_template: '{{SUITE_NAME}} | {{TEST_NAME}}'

- name: Test PyTest test import
uses: ./
if: endsWith(github.ref, 'main') == false
with:
check_name: Example Pytest Report
report_paths: test_results/python/report.xml
include_passed: true
detailed_summary: true

- name: Test Multi test import
uses: ./
Expand All @@ -43,7 +45,7 @@ jobs:
test_results/python/report.xml
summary: |-
<table><thead><tr><th> Application (src/applications) </th></tr></thead><tbody><tr><td> multi test </td></tr></tbody></table>
\n
Custom Summary
check_title_template: |-
{{SUITE_NAME}} | {{TEST_NAME}}
\n
Expand Down
2 changes: 1 addition & 1 deletion dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -108,14 +108,14 @@ export async function run(): Promise<void> {
)
}

const supportsJobSummary = process.env[exports.SUMMARY_ENV_VAR];
const supportsJobSummary = process.env['GITHUB_STEP_SUMMARY']
if (jobSummary && supportsJobSummary) {
try {
attachSummary(testResults, detailedSummary)
} catch (error) {
core.error(`❌ Failed to set the summary using the provided token. (${error})`)
}
} else if(jobSummary && !supportsJobSummary) {
} else if (jobSummary && !supportsJobSummary) {
core.warning(`⚠️ Your environment seems to not support job summaries.`)
} else {
core.info('⏩ Skipped creation of job summary')
Expand Down

0 comments on commit 3fdd534

Please sign in to comment.