-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merge RENO-4242: Update LPA Code from Main to QA #244
Conversation
Add initial github actions
Merge main
Update github actions
Move ci.yml to workflows
Add production deployment
Update package.json version to 2.0.0
Tgr production
Change "my" to "lp" as code for LPA at src/data/ilsLibraryList.ts.
RENO-4242: Update LPA Code
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
run: npm run test | ||
publish_qa: | ||
needs: test | ||
if: github.ref == 'refs/heads/tgr-qa' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update 'refs/heads/tgr-qa'
to 'refs/heads/qa'
?
aws ecs update-service --cluster nypl-library-card-app-qa --service nypl-library-card-app-qa --force-new-deployment | ||
publish_production: | ||
needs: test | ||
if: github.ref == 'refs/heads/tgr-production' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update 'refs/heads/tgr-production'
to 'refs/heads/production'
?
Requires update to Github actions prior to merge. New to be PR created. |
Change "my" to "lp" as code for LPA at src/data/ilsLibraryList.ts.