looker-open-source / components

Looker's UI Components, Design Infrastructure and more
https://components.looker.com
MIT License
62 stars 31 forks source link

chore: Release #2914

Closed mdodgelooker closed 3 years ago

mdodgelooker commented 3 years ago

[ ] stylelint errors [ ] TS errors

github-actions[bot] commented 3 years ago

Coverage report

Total coverage

Status Category Percentage Covered / Total
🟒 Statements 94.77% 7327/7731
🟑 Branches 87.11% 4670/5361
🟒 Functions 93.41% 2098/2246
🟒 Lines 95.34% 7083/7429

Status of coverage: 🟒 - ok, 🟑 - slightly more than threshold, πŸ”΄ - under the threshold

Show new covered files πŸŒ‘ ### Coverage of new files | Status | Filename | Statements | Branches | Functions | Lines | | :----: | :-------------------------------------- | :--------- | :------- | :-------- | :---- | | 🟒 | components/src/Form/Inputs/ariaProps.ts | 100% | 100% | 100% | 100% | > Status of coverage: 🟒 - ok, 🟑 - slightly more than threshold, πŸ”΄ - under the threshold
Show files with reduced coverage πŸ”» ### Reduced coverage | Status | Filename | Statements | Branches | Functions | Lines | | :----: | :--------------------------------------------------- | :------------------ | :----------------- | :------------ | :------------------ | | πŸ”΄ | design-tokens/src/utils/helpers.ts | 83.33% (-16.67% πŸ”») | 0% | 75% (-25% πŸ”») | 77.78% (-22.22% πŸ”») | | 🟒 | components/src/Form/Inputs/InputColor/InputColor.tsx | 100% | 88.89% (-1.11% πŸ”») | 100% | 100% | > Status of coverage: 🟒 - ok, 🟑 - slightly more than threshold, πŸ”΄ - under the threshold

Report generated by πŸ§ͺjest coverage report action from 0c795558ac8ac9bbbd1c62bb80b99cf42324e649

mdodgelooker commented 3 years ago

Oof all sorts of issues in this one. I've fixed some, but lint is still complaining inexplicably about a bunch of stuff in the vis packages. Aside from one unused var error, I couldn't make heads or tails of them. @dbchristopher any ideas?

dbchristopher commented 3 years ago

I'll take a look!

mdodgelooker commented 3 years ago

@dbchristopher removed the vis packages so we can figure them out separately.