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

refactor: make ref checkout for picasso scripts configurable #37

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mariajgrimaldi
Copy link
Collaborator

Description

This change makes the use of versions different than default possible, so the code available matches the version of the workflow used.

@mariajgrimaldi mariajgrimaldi requested a review from a team as a code owner November 19, 2024 09:36
This change makes the use of versions different than default possible,
so the code available matches the version of the workflow used.
@mariajgrimaldi mariajgrimaldi requested a review from a team November 25, 2024 17:32
- **Extra commands**: The workflow allows running additional custom commands with ``tutor picasso run-extra-commands``. For details, refer to the `tutor-contrib-picasso`_ documentation.
- **Environment setup**: The workflow sets up and configures Tutor Virtual Environments (TVM), installs necessary plugins like ``tutor-contrib-picasso``, and prepares the environment to build and push Docker images using the `Tutor CLI`_.
- **Environment setup**: The workflow sets up installs necessary plugins like ``tutor-contrib-picasso``, and prepares the environment to build and push Docker images using the `Tutor CLI`_.

Choose a reason for hiding this comment

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

The README still mentions TVM:

- **Environment setup**: The workflow sets up and configures Tutor Virtual Environments (TVM), installs necessary plugins like ``tutor-contrib-picasso``, and prepares the environment to build and push Docker images using the `Tutor CLI`_.

@@ -34,6 +37,9 @@ Here is an example of how to use the Picasso workflow with the inputs set:
STRAIN_PATH: redwood/base
SERVICE: mfe
ENABLE_LIMIT_BUILDKIT_PARALLELISM: false
RUNNER_WORKFLOW_LABEL: ubuntu-latest

Choose a reason for hiding this comment

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

Should we include these inputs in the README?

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