chore(overmind-devtools): ensure deps are explicitly manifested#618
chore(overmind-devtools): ensure deps are explicitly manifested#618dpraimeyuu wants to merge 2 commits intocerebral:nextfrom
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
There was a problem hiding this comment.
I think babel and webpack plug-ins should be under "devDependencies", as they are not used by the runtime but only for development.
You also should make sure that the deps are in sync with the deps in the root of the monorepo. You can check this by running npm run checkdeps from the root folder. After fix it manually as fixdeps doesn't really works as expected.
When you run npm install from the root folder of the monorepo all packages will be linked and build. Don't install dependencies in the package folders as this destroys the linking. All packages should be installed from the root folder during development.
BTW it seems there is a conflict. Make sure you start with the next branch.
|
BTW I have no idea why |
|
@henri-hulski I'll do how you advised - also documenting in the main readme (?) on how to proceed with bumping and maintaining dependencies. |
|
Sure! I think that would help maintaining overmind. |
henri-hulski
left a comment
There was a problem hiding this comment.
Hi Damian! Could you take a look at the review and fix it? Then I will merge it.
| "devtool" | ||
| ], | ||
| "dependencies": { | ||
| "@babel/plugin-transform-runtime": "^7.24.0", |
There was a problem hiding this comment.
All webpack and babel related dependencies are devDependencies not dependencies as they're only needed for development, not when you install the npm package (npm install overmind-devtools).
| "react": "^18.2.0", | ||
| "react-dom": "^18.2.0", | ||
| "react-dom": "^18.2.0", | ||
| "react-icons": "^5.0.1", |
There was a problem hiding this comment.
react-icons is ok (should be) in dependencies as it's used directly in the code.
| @@ -1,3 +1,22 @@ | |||
| # overmind-devtools | |||
| ### Introduction | |||
There was a problem hiding this comment.
We should leave # overmind-devtools so that we know where we are and underneath use ## not ###. Otherwise I get linting errors.
There was a problem hiding this comment.
Also seems like you need to rebase as there were some changes in package.json.
Btw I'm just starting to fix the start scripts. For overmin-devtools-client I was able to make the web-client work.
I also want to take a look why npm start doesn't work in this repo.
2d3ee49 to
f2d7b68
Compare
1352333 to
051354f
Compare
In this tiny PR:
package.json- local development can be started