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.

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

Add option to skip reporting when no files are changed #53

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
97160d0
Merge pull request #1 from andybelltree/filter-changed-files
andybelltree Nov 15, 2021
8676aba
Allow limiting of uncovered lines
andybelltree Nov 16, 2021
6fb6b71
Fix parameter validation
andybelltree Nov 16, 2021
df06834
Merge pull request #2 from andybelltree/limit-uncovered-lines
andybelltree Nov 16, 2021
ef9cb2c
Update README
andybelltree Nov 16, 2021
87352ea
Merge pull request #3 from andybelltree/limit-uncovered-lines
andybelltree Nov 16, 2021
9bef174
Merge pull request #4 from andybelltree/filter-changed-files
andybelltree Nov 21, 2021
1ebf368
Merge branch 'romeovs:master' into master
andybelltree Aug 29, 2022
03f4ee4
Add an option dont-post-if-no-changed-files-in-report. If this is set…
andybelltree Aug 29, 2022
4985dc8
Merge pull request #5 from andybelltree/add-skip-empty-post-option
andybelltree Aug 29, 2022
fec6f79
Add new option to action.yml
andybelltree Aug 29, 2022
172c98d
Merge pull request #6 from andybelltree/add-skip-empty-post-option
andybelltree Aug 29, 2022
b6351d3
Move substring after checking for undefined
andybelltree Aug 29, 2022
6ae0eb6
Merge pull request #7 from andybelltree/add-skip-empty-post-option
andybelltree Aug 29, 2022
978c69a
Remove summary title too if the table is empty.
andybelltree Aug 29, 2022
53cb1e0
Merge pull request #8 from andybelltree/add-skip-empty-post-option
andybelltree Aug 29, 2022
1ad7b49
Add a failure threshold for code coverage dropping
andybelltree Sep 18, 2022
be8365a
Fix some logic and add a warning to comment if dropping below threshold
andybelltree Sep 18, 2022
ce9ea1f
Fix reference to undefined variable
andybelltree Sep 19, 2022
b179652
Move threshold warning below summary
andybelltree Sep 19, 2022
42fd1ed
Merge pull request #9 from andybelltree/allow-fail-threshold-for-cove…
andybelltree Sep 19, 2022
1733fef
Upgrade to node16
andybelltree Nov 27, 2022
d98b0ef
Merge branch 'master' of https://github.com/romeovs/lcov-reporter-act…
andybelltree Nov 27, 2022
b240b9f
Merge branch 'romeovs-master'
andybelltree Nov 27, 2022
c037f0d
Add a test workflow which runs the tests
andybelltree Jul 10, 2023
3d02387
Convert to typescript
andybelltree Jul 10, 2023
f59119d
Merge pull request #11 from andybelltree/general-improvements
andybelltree Jul 10, 2023
2bf17b5
Fix an issue deleting old comments when no files have changed
andybelltree Jul 10, 2023
192152e
Go back to original API usage which was working and just change respo…
andybelltree Jul 10, 2023
d07623f
Check in main.js
andybelltree Jul 10, 2023
15dfb95
Add logging to investigate what is happening
andybelltree Jul 11, 2023
5f9a68f
Use correct login for finding existing bot comments
andybelltree Jul 11, 2023
905098a
Format document
andybelltree Jul 11, 2023
bdff45c
Merge pull request #12 from andybelltree/delete-comments-bug
andybelltree Jul 11, 2023
b94631a
Update runtime to node20
PascalMartinPhocas Feb 5, 2024
e88a462
Merge pull request #14 from PascalMartinPhocas/master
andybelltree Feb 6, 2024
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
23 changes: 23 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: Run tests

on:
push:
branches:
- main

pull_request:

jobs:
run-tests:
name: "Run tests"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: mskelton/setup-yarn@v1
with:
node-version: 20.x

- name: Build and run tests
run: |
yarn
yarn test
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,2 +1,6 @@
node_modules/
coverage/
.rollup.cache/

tsconfig.tsbuildinfo
yarn-error.log
9 changes: 9 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,15 @@ If set to true, old comments will be deleted before a new comment is posted
##### `title` (**Optional**)
If included, will be added as a title for the comment produced.

##### `max-uncovered-lines` (**Optional**)
If included, will limit the number of uncovered lines displayed in the Uncovered Lines column.

##### `dont-post-if-no-changed-files-in-report` (**Optional**)
If included, will skip posting a coverage report if no changed files would be included in the report

##### `fail-drop-percent-threshold` (**Optional**)
If included, will fail if coverage drops more than the given percentage

## Example usage

```yml
Expand Down
12 changes: 11 additions & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,16 @@ inputs:
title:
description: Title to add to the comment
required: false
max-uncovered-lines:
description: Max number of uncovered lines to display in uncovered lines column (integer)
required: false
dont-post-if-no-changed-files-in-report:
description: Set to true to prevent posting when no files in the report have been changed
required: false
default: false
fail-drop-percent-threshold:
description: If included, will fail if coverage drops more than the given percentage
required: false
runs:
using: node12
using: node20
main: dist/main.js
Loading