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

update deploy scripts to v2.0.0 #1154

Merged
merged 1 commit into from
Jul 11, 2024
Merged

update deploy scripts to v2.0.0 #1154

merged 1 commit into from
Jul 11, 2024

Conversation

elrayle
Copy link
Collaborator

@elrayle elrayle commented Jul 9, 2024

Fixes #1155


Description

The v2.0.0 deploy workflows moved many of the complex scripts to files that can be tested. This identified several bugs that when fixed allowed the service-dev to deploy successfully. The change also adds a new job to upload the package-lock.json file to the build artifacts. This is needed for the build-and-deploy job to be able to get the version from that file.

Related Work

@elrayle elrayle force-pushed the elr/ops-v1.1.1.rc1-testing branch from 6cecd59 to f8cd611 Compare July 11, 2024 05:14
@qtomlinson qtomlinson changed the title update deploy scripts to v1.1.1 update deploy scripts to v2.0.0 Jul 11, 2024
Copy link
Collaborator

@qtomlinson qtomlinson left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together!

@@ -7,8 +7,23 @@ on:
branches: [master]

jobs:
upload-package-lock-json:
name: Upload package-lock.json from this repo
runs-on: ubuntu-latest
Copy link
Collaborator

Choose a reason for hiding this comment

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

This uploading job is somewhat cumbersome. Hopefully, we can improve this in the future.

@elrayle elrayle merged commit c9419ab into master Jul 11, 2024
4 checks passed
@elrayle elrayle deleted the elr/ops-v1.1.1.rc1-testing branch July 11, 2024 21:57
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.

DEV deploy always fails
2 participants