Skip to content

Commit

Permalink
Prettier comment (#6928)
Browse files Browse the repository at this point in the history
Just making our CI workflow print a message with a suggestion for how to
fix linter issues.

See text on previous CI run, before actually fixing the issues I
introduced:
https://github.com/tensorflow/tensorboard/actions/runs/11376055312/job/31647872238
  • Loading branch information
arcra authored Oct 17, 2024
1 parent 109771f commit f57aedc
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -316,8 +316,9 @@ jobs:
with:
node-version: 16
- run: yarn install --ignore-engines
# You can run `yarn fix-lint` to fix all Prettier complaints.
- run: yarn lint
# You can run `yarn fix-lint` to fix all Prettier complaints, although at this point this will try to fix too many things.
# To fix only the files changed in this PR, see the command below.
- run: yarn lint || (( echo 'Try running `$ prettier --write $(git diff-tree --no-commit-id --name-only <latest commit> <parent branch commit> -r)`.' && false ))
# Make sure no tests are skipped with "focused" tests.
- run: |
! git grep -E 'f(it|describe)\(' 'tensorboard/*_test.ts'
Expand Down

0 comments on commit f57aedc

Please sign in to comment.