Skip to content

Correct YML syntax of the comments on example in the README #54

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

Open
facundopanizza wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Correct YML syntax of the comments on example in the README #54

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
6 changes: 3 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,12 @@ The action assumes jest configuration and jest module already present in the wor
- Do you want to fail the workflow if the commited code decreases the percentage below a tolerable level? Do you to start a healthy culture of writing test cases?
The action now also supports failing the run if percentage diff is more than a specified delta value for any file, you can specify the delta value using the variable delta
```bash
delta: 1 // the action will fail if any of the percentage dip is more than 1% for any changed file
delta: 1 # the action will fail if any of the percentage dip is more than 1% for any changed file
```

Sample workflow for running this action

```
```yml
name: Node.js CI

on: pull_request
Expand All @@ -70,7 +70,7 @@ jobs:
id: testCoverage
uses: anuraag016/Jest-Coverage-Diff@master
with:
fullCoverageDiff: false // defaults to false, if made true whole coverage report is commented with the diff
fullCoverageDiff: false # defaults to false, if made true whole coverage report is commented with the diff
runCommand: "npx jest --collectCoverageFrom='[\"src/**/*.{js,jsx,ts,tsx}\"]' --coverage --collectCoverage=true --coverageDirectory='./' --coverageReporters='json-summary' --forceExit --detectOpenHandles test/.*test.*"
delta: 0.5
```