Skip to content

improve path pattern to trigger docker build workflow#772

Merged
gonuke merged 2 commits intosvalinn:developfrom
bam241:paths_for_docker
Jul 23, 2021
Merged

improve path pattern to trigger docker build workflow#772
gonuke merged 2 commits intosvalinn:developfrom
bam241:paths_for_docker

Conversation

@bam241
Copy link
Copy Markdown
Member

@bam241 bam241 commented Jul 23, 2021

this should left out all the pieces we used for building and testing

@gonuke
Copy link
Copy Markdown
Member

gonuke commented Jul 23, 2021

needs a rebase...

Did you cancel the build's for a reason - or do we just have to assume it will work since this PR will always trigger a build anyway?

@bam241 bam241 force-pushed the paths_for_docker branch from 15e047a to 8d3cb9d Compare July 23, 2021 13:38
@bam241
Copy link
Copy Markdown
Member Author

bam241 commented Jul 23, 2021

I did cancelled it because nothing but the trigger of the workflow changed.

this one triggering the build workflow is expected as we modified .github/workflows/docker_publish.yml

@gonuke
Copy link
Copy Markdown
Member

gonuke commented Jul 23, 2021

Thanks @bam241 - we'll keep an eye on this with future PR's to see that it's behaving as expected

@gonuke gonuke merged commit f03433e into svalinn:develop Jul 23, 2021
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