Skip to content

Commit

Permalink
ci: remove current attempts at semantic-release
Browse files Browse the repository at this point in the history
  • Loading branch information
ianwremmel committed Feb 12, 2023
1 parent 4619453 commit 9c72890
Show file tree
Hide file tree
Showing 6 changed files with 112 additions and 1,756 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ jobs:
run: |
git config user.name "${GITHUB_ACTOR}"
git config user.email "${GITHUB_ACTOR}@users.noreply.github.com"
- run: npx nx run-many --target=semantic-release
- run: echo WIP && exit 1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
17 changes: 6 additions & 11 deletions .husky/pre-commit
Original file line number Diff line number Diff line change
Expand Up @@ -17,16 +17,11 @@ if ! git diff --staged --exit-code README.md; then
git add README.md
fi

# Don't run in CI since it sem-rel may make non-conformant commits (that never
# get pushed)
if [ -z ${CI+x} ]; then
# Autofix files that nx can autofix
if ! npx nx format:check; then
echo 'nx format:check has detected files that will fail CI checks'
echo 'Please run `npx nx format:write` to fix them'
echo 'Ideally, this pre-commit hook could determine which files they are, but '
echo 'nx format only indicates there are failing files, not which ones.'
if ! npx nx format:check; then
echo 'nx format:check has detected files that will fail CI checks'
echo 'Please run `npx nx format:write` to fix them'
echo 'Ideally, this pre-commit hook do that fix for you, but there is no way'
echo 'to know which files were effected and pass them to `git add`'

exit 1
fi
exit 1
fi
4 changes: 0 additions & 4 deletions nx.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,6 @@
"build": {
"dependsOn": ["^build"],
"inputs": ["default", "^dist"]
},
"semantic-release": {
"dependsOn": ["build", ",^semantic-release"],
"inputs": ["default", "dist"]
}
},
"tasksRunnerOptions": {
Expand Down
Loading

0 comments on commit 9c72890

Please sign in to comment.