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
Bugfix/loading box dependency #189
Conversation
eslint needs to be configured to pick up exports that aren't in package.json see import-js/eslint-plugin-import#1049 In the meantime we are including @govuk-react/storybook as a dependency, which will help ensure subdependencies are included.
@@ -33,6 +33,7 @@ | |||
"@govuk-react/related-items": "^0.1.25", | |||
"@govuk-react/search-box": "^0.1.25", | |||
"@govuk-react/select": "^0.1.25", | |||
"@govuk-react/storybook": "^0.1.25", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you haven't added @govuk-react/loading-box
to this file...
"@govuk-react/label-text": "^0.1.25", | ||
"@govuk-react/list-item": "^0.1.25", | ||
"@govuk-react/list-navigation": "^0.1.25", | ||
"@govuk-react/multi-choice": "^0.1.25", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
think we should add in loading-box
here
Codecov Report
@@ Coverage Diff @@
## master #189 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 44 44
Lines 299 299
Branches 8 8
=====================================
Hits 299 299 Continue to review full report at Codecov.
|
@govuk-react/loading-box
wasn't a sub dependency of govuk-react. We want issues like this to be picked up automatically.eslint needs to be configured to pick up exports that aren't in package.json, but this can't be done right now, see import-js/eslint-plugin-import#1049.
In the meantime we are including @govuk-react/storybook as a dependency, which will help ensure the storybook sub dependencies are included.
Also: