Skip to content

"release" pipeline : scda.test reverse dependency + hotfix + rhub #248

Merged
munoztd0 merged 36 commits intoCRANfrom
scda_test_reversedependency
Apr 8, 2026
Merged

"release" pipeline : scda.test reverse dependency + hotfix + rhub #248
munoztd0 merged 36 commits intoCRANfrom
scda_test_reversedependency

Conversation

@munoztd0
Copy link
Copy Markdown
Collaborator

@munoztd0 munoztd0 commented Mar 25, 2026

Pull Request

fix:

[Feature Request]: scda.test revdep #243

and

[Feature Request]: Hotfix Pipeline #242

This is the proof that the hotfix file is source properly -> https://github.com/johnsonandjohnson/junco/actions/runs/23743976032/job/69167713917?pr=250

@munoztd0 munoztd0 changed the title "release" pipepline : scda.test reverse dependency + hotfix + rhub "release" pipeline : scda.test reverse dependency + hotfix + rhub Mar 27, 2026
@munoztd0 munoztd0 requested review from eanokian and gmbecker March 30, 2026 10:09
required: false
push:
branches:
- 'feature/hotfix-*'
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I did basic tests on the hotfix pipeline and it's working as expected.
This regexp only matches branch names starting with "feature/hotfix-xxx"; I'm wondering if this naming is too specific. It doesn't match branch names like "hotfix-xxx".
Just a minor comment and I'm not sure if there's already a naming convention for hotfix branches.

Copy link
Copy Markdown
Collaborator

@eanokian eanokian left a comment

Choose a reason for hiding this comment

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

LGTM, I just left a minor comment

@munoztd0 munoztd0 merged commit 7df0af0 into CRAN Apr 8, 2026
28 of 29 checks passed
@munoztd0 munoztd0 deleted the scda_test_reversedependency branch April 8, 2026 10:47
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