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.

Sign up for GitHub

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

Jump to bottom

Make CI and deployment.yaml share the same dependency install script #190

Merged
merged 3 commits into from
Aug 31, 2023
Merged

Make CI and deployment.yaml share the same dependency install script #190

merged 3 commits into from
Aug 31, 2023

Conversation

ChrisLovering
Copy link
Member

No description provided.

@ChrisLovering ChrisLovering force-pushed the share-deployment-dep-script-with-ci branch from bf7fb5d to 4e58eb5 Compare August 30, 2023 22:09
@coveralls
Copy link

coveralls commented Aug 30, 2023 •
edited
Loading

Coverage Status

coverage: 90.947%. remained the same when pulling f637e54 on share-deployment-dep-script-with-ci into 25b84f4 on main.

@ChrisLovering ChrisLovering force-pushed the share-deployment-dep-script-with-ci branch 3 times, most recently from ddb35ad to ac4f761 Compare August 30, 2023 22:54
@ChrisLovering ChrisLovering force-pushed the share-deployment-dep-script-with-ci branch from ac4f761 to 9586c5b Compare August 30, 2023 22:55
@ChrisLovering ChrisLovering force-pushed the share-deployment-dep-script-with-ci branch from 38a8e97 to f637e54 Compare August 31, 2023 09:09
MarkKoz
MarkKoz approved these changes Aug 31, 2023
View reviewed changes
Copy link
Member

@MarkKoz MarkKoz left a comment

Choose a reason for hiding this comment

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

Thanks

@ChrisLovering ChrisLovering merged commit 7303301 into main Aug 31, 2023
7 checks passed
@ChrisLovering ChrisLovering deleted the share-deployment-dep-script-with-ci branch August 31, 2023 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarkKoz MarkKoz MarkKoz approved these changes

@jb3 jb3 Awaiting requested review from jb3 jb3 is a code owner

@Den4200 Den4200 Awaiting requested review from Den4200 Den4200 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ChrisLovering @coveralls @MarkKoz