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

Unzip ibsolution file #3

Merged
merged 15 commits into from
May 9, 2024
Merged

Unzip ibsolution file #3

merged 15 commits into from
May 9, 2024

Conversation

hannahflroiter
Copy link
Collaborator

If download_ibsolution flag is set then also store the unzipped version of the ibsolution. If this is stored as part of a CI pipeline then version control of the code will be possible

Also:

  • Set write_to_local to False when downloading ibsolution to upload to target environment
  • Added dotenv for easier development when running locally

@@ -15,6 +15,10 @@
from migration_helpers import download_ibsolution, compile_and_package_ib_solution, \
download_dependencies_from_dev_and_upload_to_prod

from dotenv import load_dotenv
Copy link
Collaborator

Choose a reason for hiding this comment

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

What does load_dotenv do here?

Copy link
Collaborator Author

@hannahflroiter hannahflroiter May 1, 2024

Choose a reason for hiding this comment

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

It will load in any environment variables stored in a .env file and if it doesn't exist does nothing, I've added a bit to the readme to explain

cicd/migration_helpers.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@BNJHope BNJHope left a comment

Choose a reason for hiding this comment

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

Looks good, added a couple of refactoring components/questions

cicd/migration_helpers.py Outdated Show resolved Hide resolved
cicd/migration_helpers.py Outdated Show resolved Hide resolved
cicd/migration_helpers.py Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
Copy link
Collaborator

@BNJHope BNJHope left a comment

Choose a reason for hiding this comment

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

LGTM - small nit but approved 🙌

cicd/migration_helpers.py Outdated Show resolved Hide resolved
hannahflroiter and others added 2 commits May 9, 2024 12:42
Co-authored-by: Ben Hope <bnjhope1@gmail.com>
@hannahflroiter hannahflroiter merged commit b9b20bd into main May 9, 2024
1 check passed
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.

3 participants