Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add node_modules to the RPM. #835

Merged
merged 7 commits into from
Mar 22, 2019
Merged

Conversation

jpwhite4
Copy link
Member

@jpwhite4 jpwhite4 commented Mar 1, 2019

node_modules dependencies are now packaged in the RPM. There will be a corresponding update to the supremm RPM to account for this.

This change includes a %pre hook in the rpm that saves the (non-rpm managed) node_modules directory that could be present on an upgrade. This %pre hook should be removed by the next release. I have included a defensive check to ensure that the save code only runs when the version of the rpm is 8.1.0

@jpwhite4
Copy link
Member Author

jpwhite4 commented Mar 1, 2019

Need to update the included packages list in the documentation

jpwhite4 added a commit to jpwhite4/xdmod-supremm that referenced this pull request Mar 4, 2019
plessbd
plessbd previously approved these changes Mar 4, 2019
@jpwhite4 jpwhite4 added this to the 8.1.0 milestone Mar 14, 2019
@jpwhite4 jpwhite4 merged commit 81237e4 into ubccr:xdmod8.1 Mar 22, 2019
@jpwhite4 jpwhite4 deleted the node_modules branch March 22, 2019 19:40
@plessbd plessbd added the enhancement Enhancement of the functionality of an existing feature label Mar 23, 2019
jpwhite4 added a commit to jpwhite4/xdmod-supremm that referenced this pull request Mar 25, 2019
jpwhite4 added a commit to ubccr/xdmod-supremm that referenced this pull request Mar 26, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement Enhancement of the functionality of an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants