Skip to content

Conversation

@Blendify
Copy link
Member

NPM made some changes to package.json and package-lock. From what I can
tell these changes are backward compatible, see
https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json

I also moved dependencies that affect the build package to regular
dependencies instead of devDependencies. As far as I know this is
correct usage and allows developers to easily see what affets the built
backage and what affects the devOps process.

NPM made some changes to package.json and package-lock. From what I can 
tell these changes are backward compatible, see 
https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json

I also moved dependencies that affect the build package to regular 
dependencies instead of devDependencies. As far as I know this is 
correct usage and allows developers to easily see what affets the built 
backage and what affects the devOps process.
Copy link
Member

@stsewd stsewd left a comment

Choose a reason for hiding this comment

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

great! we may hold to merge this as well for a bigger release

@stsewd stsewd added the Status: blocked Issue is blocked on another issue label Mar 1, 2021
@Blendify Blendify added this to the 1.0.0 milestone Mar 11, 2021
@Blendify
Copy link
Member Author

This is not a user affecting change so it shouldn't matter if it's a big or small release. I do want to break this into two patches though...

Copy link
Collaborator

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

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

This looks entangled with removing Wyrm, so would have to be on a later release. The other PR that touches this file probably is more close to merging, but this one has to be pushed pretty far out.

@agjohnson agjohnson modified the milestones: 1.0.0, 2.0 Mar 12, 2021
@Blendify
Copy link
Member Author

This is not related to removing wyrm, the splitting of dev requirements are also not necessary, I meant to update this patch.

"url": "https://github.com/readthedocs/sphinx_rtd_theme/issues"
},
"homepage": "https://github.com/readthedocs/sphinx_rtd_theme",
"dependencies": {},
Copy link
Member Author

Choose a reason for hiding this comment

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

npm7 doesnt like this being empty.

@Blendify Blendify removed the Status: blocked Issue is blocked on another issue label Mar 20, 2021
@Blendify Blendify requested a review from a team as a code owner January 19, 2022 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants