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

Enhance Workflow with Detailed Summaries and Improved Deployment #11

Merged
merged 6 commits into from
Jul 27, 2024
Merged

Conversation

maxisoft
Copy link
Owner

Description

This pull request includes the following changes:

  1. feat: enhance workflow with detailed step summaries and improved deployment

    • Added step summaries to provide detailed information in the workflow summary
    • Updated conditional checks and environment variable handling for better clarity
    • Improved cleanup steps to ensure temporary files and directories are removed
    • Corrected the name of the step for pushing the kernels directory to another repository
  2. fix: update main.py to use environment variable for include_private

    • Added os import to main.py
    • Changed default value of include_private to False
    • Updated main function to use environment variable KAGGLE_KERNELS_PRIVATE for include_private parameter

These enhancements improve the clarity and functionality of the workflow by adding detailed step summaries and ensuring proper handling of environment variables.

- Added instructions to set the ARCHIVE_PASSWORD secret for artifact encryption
- Added conditional check for ARCHIVE_PASSWORD secret
- Implemented high compression and password protection for kernels archive
- Updated artifact upload to use the new kernels.7z file
- Added conditional steps for deploying to another repository if SSH_DEPLOY_KEY and DESTINATION_REPOSITORY_NAME are set
- Improved cleanup steps to remove temporary files and directories
- Added os import to main.py
- Changed default value of include_private to False
- Updated main function to use environment variable KAGGLE_KERNELS_PRIVATE for include_private parameter
…oyment

- Added step summaries to provide detailed information in the workflow summary
- Updated conditional checks and environment variable handling for better clarity
- Improved cleanup steps to ensure temporary files and directories are removed
- Corrected the name of the step for pushing the kernels directory to another repository
@maxisoft maxisoft self-assigned this Jul 27, 2024
# Conflicts:
#	.github/workflows/doit.yml
@maxisoft maxisoft merged commit 5918dce into main Jul 27, 2024
2 of 3 checks 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.

1 participant