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

FEAT: Workflow for testing corpus PRs #473

Open
wants to merge 2 commits into
base: integration
Choose a base branch
from

Conversation

carlwilson
Copy link
Contributor

@carlwilson carlwilson commented Jul 12, 2023

  • initial workflow just to detect and list any changes to corpus files.

An example of the workflow in action can be seen here: https://github.com/opf-labs/eark-ip-test-corpus/actions/runs/5531486888/jobs/10092244748#step:4:1 which I can use to zero in on changes. Next stop add a validation phase.

- initial workflow just to detect and list any changes to corpus files.
@carlwilson carlwilson added the automation GitHub Actions Automation label Jul 12, 2023
@carlwilson carlwilson self-assigned this Jul 12, 2023
@PhillipAasvangTommerholt PhillipAasvangTommerholt requested review from dockmd and removed request for scfkmd February 22, 2024 12:41
Copy link
Collaborator

@dockmd dockmd left a comment

Choose a reason for hiding this comment

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

Must be fixed if we need this workflow

id: changed-corpora-files
uses: tj-actions/changed-files@v37
with:
files: corpora/**
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think we should merge this PR since there is no 'corpora' folder anymore. I will use a similar approach to retrieve changed files in workflow that validates testcases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation GitHub Actions Automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants