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

[Actions] GitHub Actions will now also built upwards dependencies but they won't block merging #1496

Merged
merged 11 commits into from
Jan 16, 2024

Conversation

VeithMetro
Copy link
Contributor

@VeithMetro VeithMetro commented Jan 11, 2024

This is an initial pull request - the new workflows have to be called from my branches for now (there is a dependency between the workflows), since we need to first merge new templates in each repo. Then, I'll make one more set of pull requests to redirect this new feature to master branches of our repos.

Required checks got updated to the new ones by Stephen.

sebaszm
sebaszm previously approved these changes Jan 11, 2024
@VeithMetro VeithMetro changed the title [Actions] GitHub Actions will now also built upwards dependencies (not required to merge) [Actions] GitHub Actions will now also built upwards dependencies but they won't block merging Jan 11, 2024
@VeithMetro VeithMetro merged commit 3072f62 into rdkcentral:master Jan 16, 2024
34 checks passed
@VeithMetro VeithMetro deleted the development/actions branch February 2, 2024 09:27
bramoosterhuis pushed a commit to bramoosterhuis/Thunder that referenced this pull request Apr 26, 2024
… they won't block merging (rdkcentral#1496)

* Create Linux build template

* Use the Linux build template

* Building additional repos in separate jobs as not required checks

* Checking if non required actions won't block the merge

* Checking if commenting works as intended

* Final prep for a merge

* Apparently the new workflow cannot be initialized before merging

So it seems I have to use the one from my branch in this initial pull requests..

* Trying to enable other builds

---------

Co-authored-by: Pierre Wielders <pierre@wielders.net>
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.

3 participants