Skip to content

Triggering testing workflows when there are changes on the workflows#1320

Open
pacostas wants to merge 1 commit intopaketo-buildpacks:mainfrom
pacostas:trigger-workflows-on-changes
Open

Triggering testing workflows when there are changes on the workflows#1320
pacostas wants to merge 1 commit intopaketo-buildpacks:mainfrom
pacostas:trigger-workflows-on-changes

Conversation

@pacostas
Copy link
Member

@pacostas pacostas commented Mar 6, 2026

Summary

This PR adds a hook in order to trigger the workflows while there are changes on the workflow itself

Use Cases

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@pacostas pacostas requested review from a team as code owners March 6, 2026 10:07
@pacostas pacostas force-pushed the trigger-workflows-on-changes branch from 12a1de4 to da945f7 Compare March 6, 2026 10:27
@pacostas
Copy link
Member Author

pacostas commented Mar 6, 2026

@paketo-buildpacks/java-maintainers The java sample integration tests seems like they are failing.

@dmikusa
Copy link
Contributor

dmikusa commented Mar 9, 2026

@pacostas There is a script that runs for the Java integration tests. The script looks at the changes and determines which samples to run for the integration test. Because there are so many Java samples, we don't want to run all the samples if a change was isolated to just one sample.

This change is breaking that script. I'm not sure why offhand, but I'd suspect it's based on the inputs changing. There's likely an assumption about the inputs will be under the java/ section, but this means they could be under .github/.

The easiest thing is to add set -x into the bash code, so we can see what it's doing.

That bash will likely need adjusted to work with this new scenario.

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