Skip to content

Update Workflows to Version 1.0.0#114

Merged
K-Meech merged 1 commit into
mainfrom
update/workflows
Apr 17, 2026
Merged

Update Workflows to Version 1.0.0#114
K-Meech merged 1 commit into
mainfrom
update/workflows

Conversation

@K-Meech

@K-Meech K-Meech commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

🤖 This is an automated build

Update Workflows from sandpaper version 0.18.5 -> 1.0.0

@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

ℹ️ Modified Workflows

This pull request contains modified workflow files and no preview will be created.

Workflow files modified:

  • .github/workflows/README.md
  • .github/workflows/docker_build_deploy.yaml
  • .github/workflows/sandpaper-version.txt
  • .github/workflows/update-workflows.yaml
  • .github/workflows/workflows-version.txt

If this is not from a trusted source, please inspect the changes for any malicious content.

@K-Meech
K-Meech requested a review from thompson318 March 13, 2026 09:06
@thompson318

Copy link
Copy Markdown
Contributor

Update Workflows from sandpaper version 0.18.5 -> 1.0.0

I think this should be sandpaper version 0.18.5 -> https://github.com/carpentries/workbench-workflows v 1.0.0

@thompson318 thompson318 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The workflow updates look sensible enough. Suggest we merge and see what happens. After that we should double check that the instructions in our README still work.

push:
branches:
- 'main'
- 'l10n_main'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't know what the l10n_main branch is for. Couldn't find any reference to it in the documentation.

Suggested change
- 'l10n_main'

@K-Meech

K-Meech commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for reviewing this @thompson318 - I realise I never got around to merging this! I'm also not sure what the 'l10n_main' branch refers to, but as these are automated updates coming directly from Carpentries - it shouldn't cause any problems.

I'll go ahead and merge now 👍

@K-Meech
K-Meech merged commit 8acfb64 into main Apr 17, 2026
6 checks passed
@K-Meech
K-Meech deleted the update/workflows branch April 17, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants