-
Notifications
You must be signed in to change notification settings - Fork 24
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
Chore: Move presentational components to design system package #155
Conversation
30cbb82
to
49f902b
Compare
const HtmlWebpackPlugin = require('html-webpack-plugin'); | ||
const CopyPlugin = require('copy-webpack-plugin'); | ||
const WebpackBar = require('webpackbar'); | ||
const commonConfig = require('../../webpack.shared.cjs'); |
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.
👍
This was pending for sometime
"cli:dev": "tsc-watch", | ||
"cli:build": "tsc --build", | ||
"cli:dev": "npm run dev --workspace=@cookie-analysis-tool/cli", | ||
"cli:build": "npm run build --workspace=@cookie-analysis-tool/cli", |
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.
👍
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.
Looks good to me
Description
This PR moves presentational components into its own package named design system. In conjunction to that some config files are made more modular in preparation for cli report package.
Relevant Technical Choices
Testing Instructions
Additional Information:
Screenshot/Screencast
Fixes #