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.

Sign up for GitHub

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

Jump to bottom

OSOE-593: Add job summary with tool versions #223

Merged
merged 17 commits into from
Apr 14, 2023
Merged

OSOE-593: Add job summary with tool versions #223

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4e4d140
Adding printing of config summary to workflow
Apr 1, 2023
fcc427c
Add missing step title
Apr 1, 2023
3b7c044
Trying to get the job summary printed
Apr 1, 2023
aa58a62
Try using the action again
Apr 3, 2023
52bc532
Use markdown headers
Apr 3, 2023
b5d8a0a
Trying default shell
Apr 3, 2023
807b48f
Add missing shell
Apr 3, 2023
2b6130b
Switching to pwsh
Apr 3, 2023
77b393d
Using a table
Apr 3, 2023
0b50fbb
Restore build, run summary always
Apr 3, 2023
812a84b
Updating tool version table formatting
0liver Apr 4, 2023
35f177f
Don't print the config summary in case the workflow is canceled by ot…
0liver Apr 4, 2023
edf3902
Merge remote-tracking branch 'origin/dev' into issue/OSOE-593
Apr 6, 2023
c1f7e0f
Merge remote-tracking branch 'origin/dev' into issue/OSOE-593
Apr 7, 2023
daa35f7
Merge remote-tracking branch 'origin/dev' into issue/OSOE-593
Apr 11, 2023
bf3bbab
Merge branch 'dev' into issue/OSOE-593
Apr 11, 2023
a151493
Targeting dev in workflow
dministro Apr 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions .github/actions/print-config-summary/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
name: Print summary of important configuration
description: Contains .NET, Node.js, and pnpm versions, among others

runs:
using: "composite"
steps:
- name: Write configuration to summary
shell: pwsh
run: |
$toolVersions = @"
| Tool | Version |
| :--- | ------: |
| dotnet | $(dotnet --version) |
| node | $(node -v) |
| pnpm | $(pnpm -v) |
"@
$toolVersions >> $env:GITHUB_STEP_SUMMARY
6 changes: 5 additions & 1 deletion .github/workflows/build-and-test-orchard-core.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ jobs:
with:
directory: ${{ inputs.build-directory }}

- name: Enable Node corepack
- name: Enable Node.js corepack
uses: Lombiq/GitHub-Actions/.github/actions/enable-corepack@dev

- name: Build and Static Code Analysis
Expand All @@ -213,6 +213,10 @@ jobs:
binary-log-artifact-retention-days: ${{ inputs.build-binary-log-artifact-retention-days }}
warnings-as-errors: ${{ inputs.warnings-as-errors }}

- name: Print configuration summary
dministro marked this conversation as resolved.
Show resolved Hide resolved
if: always()
0liver marked this conversation as resolved.
Show resolved Hide resolved
uses: Lombiq/GitHub-Actions/.github/actions/print-config-summary@issue/OSOE-593
dministro marked this conversation as resolved.
Show resolved Hide resolved

- name: Set up SQL Server
if: inputs.set-up-sql-server == 'true'
uses: Lombiq/GitHub-Actions/.github/actions/setup-sql-server@dev
Expand Down