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

Optimize Dockerfile. #1073

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Optimize Dockerfile. #1073

merged 2 commits into from
Oct 6, 2023

Conversation

mariacarmina
Copy link
Member

Description

Optimize Dockerfile for further deployments.

Is this PR related with an open issue?

Related to Issue #1055

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • Follows the code style of this project.
  • Tests Cover Changes
  • Documentation

@mariacarmina mariacarmina self-assigned this Oct 4, 2023
@mariacarmina
Copy link
Member Author

mariacarmina commented Oct 4, 2023

For the moment, needs further investigation:

oceanprotocol/aquarius             test      7c99925aaaad   50 seconds ago   493MB

Before:

oceanprotocol/aquarius             test-before   ca923a4b07b5   About a minute ago   549MB

@codeclimate
Copy link

codeclimate bot commented Oct 5, 2023

Code Climate has analyzed commit a228c88 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 84.4% (0.2% change).

View more on Code Climate.

@mariacarmina
Copy link
Member Author

oceanprotocol/aquarius             test-decrease       cbc8d3dcc2be   2 minutes ago       455MB

@mariacarmina mariacarmina merged commit 2c16bd8 into main Oct 6, 2023
5 checks passed
@mariacarmina mariacarmina deleted the optimise-dockerfile branch October 6, 2023 11:40
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.

2 participants