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

Initialize actions #1

Merged
merged 36 commits into from
Jul 4, 2024
Merged

Initialize actions #1

merged 36 commits into from
Jul 4, 2024

Conversation

pawelru
Copy link
Contributor

@pawelru pawelru commented Jun 27, 2024

No description provided.

walkowif and others added 19 commits June 27, 2024 11:36
Fix
This reverts commit aa52eb3.
walkowif and others added 2 commits June 27, 2024 18:08
Co-authored-by: André Veríssimo <[email protected]>
Signed-off-by: Pawel Rucki <[email protected]>
Copy link
Contributor

@averissimo averissimo left a comment

Choose a reason for hiding this comment

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

Works well with my testing after a few bugs were corrected (added as single comments so that they could be easily merged).

Great job leveraging pkgdepends for the bulk of the work 💯

I used the following 4 packages to test this out

averissimo/pkg2 (custom branch)
| averissimo/pkg1 (only main branch)
| averissimo/pkg3 (custom branch)
| pkg4=averissimo/r-pkg4 (custom branch)

Namely on averissimo/pkg2 with r-hub action

  • (with a test that only works with the exact combination of branches)

I noticed that this doesn't yet support composed feature branches such as feat2@feat1. Are you planning on it?

pawelru added 8 commits June 28, 2024 12:45
@pawelru pawelru marked this pull request as ready for review July 3, 2024 15:25
pawelru added 4 commits July 3, 2024 17:28
Copy link
Contributor

@cicdguy cicdguy left a comment

Choose a reason for hiding this comment

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

Can be refactored later. Should be pretty good for v1

@pawelru pawelru merged commit 0d8c7b4 into main Jul 4, 2024
2 checks passed
@pawelru pawelru deleted the initialize-actions branch July 4, 2024 07:31
@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants