Skip to content

Conversation

neoformit
Copy link
Collaborator

The workflow that tests changes to labs and posts labs links has stopped working because of some change in GitHub workflows. I think this should fix it, and also includes a test for the workflow which uses act to run the workflow in docker container.

@neoformit neoformit requested review from Copilot and paulzierep August 4, 2025 06:03
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a broken GitHub Actions workflow for testing lab page changes and adds comprehensive testing capabilities using the act tool to run workflows locally in Docker containers.

  • Simplifies the workflow by using a single checkout with full git history instead of multiple separate checkouts
  • Adds robust testing infrastructure with an isolated test script that validates workflow behavior
  • Includes configuration for local workflow testing with Docker

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/labs_test.yml Streamlines workflow logic to use single checkout and improves git diff handling for PR changes
.github/workflows/tests/test_labs_test.sh Comprehensive test script that creates isolated repositories and validates workflow artifacts
.actrc Configuration file specifying Docker image for local GitHub Actions testing

Copy link
Member

@bebatut bebatut left a comment

Choose a reason for hiding this comment

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

Thanks @neoformit

Copy link
Member

Choose a reason for hiding this comment

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

Could you put this script in the sources folder? Thanks a lot!

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