Skip to content

Commit

Permalink
[Issue #2029] Re-add items from merge commit (#2178)
Browse files Browse the repository at this point in the history
## Summary
Fixes #2029

### Time to review: __5 mins__

## Changes proposed
This commit was necessary b/c the changes had conflicts and required a
merge commit. The merge commit blocked the ability to "rebase and merge"
the history from the nava fork. This re-adds those updates:
22562ce
  • Loading branch information
acouch committed Sep 19, 2024
1 parent dd39045 commit 23f14d4
Show file tree
Hide file tree
Showing 6 changed files with 10,703 additions and 17,720 deletions.
2 changes: 1 addition & 1 deletion api/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,4 @@ coverage.*
/test-results/

# localstack
/volume
/volume
2 changes: 2 additions & 0 deletions frontend/.eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,8 @@ module.exports = {
"@typescript-eslint/no-explicit-any": "error",
// Just warn since playwright tests may not use screen the way jest would
"testing-library/prefer-screen-queries": "warn",
// Prevent unnecessary console statements
"no-console": ["error", { allow: ["warn", "error"] }],
},
},
],
Expand Down
4 changes: 4 additions & 0 deletions frontend/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -51,3 +51,7 @@ npm-debug.log*
/playwright-report/
/blob-report/
/playwright/.cache/

# pa11y
screenshots-output/*
pa11y_output.txt
Loading

1 comment on commit 23f14d4

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report for ./frontend

St.
Category Percentage Covered / Total
🟢 Statements 90.6% 790/872
🟡 Branches 71.79% 224/312
🟢 Functions 87.89% 167/190
🟢 Lines 91.27% 742/813

Test suite run success

178 tests passing in 54 suites.

Report generated by 🧪jest coverage report action from 23f14d4

Please sign in to comment.