Skip to content

Commit

Permalink
fix: rebuild dist/ with latest tsc (#51)
Browse files Browse the repository at this point in the history
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
  • Loading branch information
Two-Hearts committed Jan 16, 2024
1 parent 46fc417 commit a81f690
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 8 deletions.
17 changes: 13 additions & 4 deletions .github/workflows/check-dist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,27 +17,36 @@ on:
push:
paths-ignore:
- '**.md'
- 'CODEOWNERS'
- 'MAINTAINERS'
pull_request:
paths-ignore:
- '**.md'
- 'CODEOWNERS'
- 'MAINTAINERS'
workflow_dispatch:

jobs:
check-dist:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Setup Node 16.x
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: 16.x
cache: npm
- name: Show versions
run: |
echo node js version: $(node -v)
echo npm version: $(npm --version)
echo latest tsc version: $(tsc --version)
- name: Rebuild the dist/ directory
run: npm run build
- name: Compare the expected and actual dist/ directories
run: |
if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
echo "DIFFERENCES DETECTED: 'npm run build' is needed after code changes. See status below:"
git diff
echo "DIFFERENCES DETECTED: 'npm update -g typescript' and 'npm run build' are needed after code changes. See status below:"
git diff --ignore-space-at-eol dist/
exit 1
fi
2 changes: 1 addition & 1 deletion dist/lib/install.js.map

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

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

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

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

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

Loading

0 comments on commit a81f690

Please sign in to comment.