Skip to content
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

Build : Bundle package-lock.json in developper release (and commit to git tag) #4686

Closed
OlivierJaquemet opened this issue Nov 29, 2018 · 4 comments

Comments

@OlivierJaquemet
Copy link

OlivierJaquemet commented Nov 29, 2018

Do you want to request a feature or report a bug?
Build feature

What is the current behavior?
When an official release is created, the developper zip archive contains the package.json used at the build time. However, there it does not provides indication of which "real" versions were used by npm when dependencies were resolved.

What is the expected behavior?
Adding the package-lock.json file to the developper zip archive, and to the git source tree for the corresponding tag, would add many advantages as described in the official NPM package-lock.json documentation.
Notably "time travel" related :

  • easier way to re-build a consistent release for a specific version.
  • track dependencies (and possibly vulnerable ones)
    ...
@spocke
Copy link
Member

spocke commented Dec 3, 2018

Will discuss this internally would be nice to be able to use npm or yarn lock files to install the dependencies needed for a particular release. We have avoided committing these since we do rapid development of all dependencies we use @latest on everything internal but it would be nice to be able to get the state back to a particular release.

@OlivierJaquemet
Copy link
Author

Thanks for the feedback @spocke

@lnewson
Copy link
Contributor

lnewson commented Mar 1, 2019

Just wanted to let you know, that as of 5.0.2 we'll now be bundling both package-lock.json and yarn.lock files in the developer distribution downloads. You should be able to see that the nightly builds already include the lock files: http://download.tiny.cloud/tinymce/community/tinymce_nightly_dev.zip

@OlivierJaquemet
Copy link
Author

Thanks. I guess I could close this issue, however it would not have the propre labels (5.x)... whatever

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants