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.

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

Enable coverage reporting #27

Merged
merged 6 commits into from
Mar 15, 2024
Merged

Enable coverage reporting #27

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
983347d
build(ava): Migrate to watchMode.ignoreChanges config
flovogt Mar 14, 2024
23bd16b
build(ava): Adjust coverage thresholds
flovogt Mar 14, 2024
6725cf7
ci(ci): Enable coveralls coverage reporting
flovogt Mar 14, 2024
853e311
style(ava): Add missing trailing comma
flovogt Mar 14, 2024
93fc70d
build(ava): Adjust branch coverage threshold
flovogt Mar 14, 2024
c5fa25a
docs(README.md): Add coverage badge
flovogt Mar 14, 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
15 changes: 11 additions & 4 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,20 @@ jobs:
- name: Install dependencies
run: npm ci

- name: Perfrom ESLint check
run: npm run lint

- name: Perfrom Licenses check
run: npm run check-licenses

- name: Perform dependency check
run: npm run depcheck

- name: Perform checks and tests
run: npm test
env:
NODE_OPTIONS: ""
- name: Perform build
run: npm run build-test

- name: Run unit tests
run: npm run coverage

- name: Send report to Coveralls for package @ui5/linter
uses: coverallsapp/[email protected]
6 changes: 3 additions & 3 deletions .nycrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
"reporter": ["lcov", "text", "text-summary"],
"include": ["src/**"],
"check-coverage": true,
"statements": 80,
"branches": 75,
"statements": 72,
"branches": 61,
"functions": 75,
"lines": 80,
"lines": 72,
"watermarks": {
"statements": [70, 90],
"branches": [70, 90],
Expand Down
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@

> A static code analysis tool for UI5

<!-- [![Coverage Status](https://coveralls.io/repos/github/SAP/ui5-linter/badge.svg)](https://coveralls.io/github/SAP/ui5-linter) -->
[![OpenUI5 Community Slack (#tooling channel)](https://img.shields.io/badge/slack-join-44cc11.svg)](https://ui5-slack-invite.cfapps.eu10.hana.ondemand.com/)
[![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-v2.1%20adopted-ff69b4.svg)](https://github.com/SAP/ui5-linter?tab=coc-ov-file#readme)
[![REUSE status](https://api.reuse.software/badge/github.com/SAP/ui5-linter)](https://api.reuse.software/info/github.com/SAP/ui5-linter)
[![Coverage Status](https://coveralls.io/repos/github/SAP/ui5-linter/badge.svg)](https://coveralls.io/github/SAP/ui5-linter)

## Description

Expand Down
10 changes: 6 additions & 4 deletions ava.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@ export default {
files: [
"test/lib/**/*.ts",
],
ignoredByWatcher: [
"test/tmp/**",
"lib/**",
],
watchMode: {
ignoreChanges: [
"test/tmp/**",
"lib/**",
],
},
nodeArguments,
workerThreads: false,
};