Skip to content

scm: add SCMType enum to helpers (bug 2019139)#945

Closed
shtrom wants to merge 1 commit intomainfrom
bug1979247/try/patches-SCMType
Closed

scm: add SCMType enum to helpers (bug 2019139)#945
shtrom wants to merge 1 commit intomainfrom
bug1979247/try/patches-SCMType

Conversation

@shtrom
Copy link
Member

@shtrom shtrom commented Feb 24, 2026

This is a clean split of the SCMType work from Bug 1979247, to land separately.

@github-actions
Copy link

View this pull request in Lando to land it once approved.

@shtrom shtrom marked this pull request as ready for review February 24, 2026 03:51
@shtrom shtrom requested a review from a team as a code owner February 24, 2026 03:51
@shtrom shtrom force-pushed the bug1979247/try/patches-SCMType branch 2 times, most recently from f9df6a3 to d9ed18b Compare February 24, 2026 05:25
@shtrom shtrom force-pushed the bug1979247/try/patches-SCMType branch from d9ed18b to a790407 Compare February 24, 2026 07:41
Copy link
Member

@cgsheeh cgsheeh left a comment

Choose a reason for hiding this comment

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

LGTM, but this needs a bug before it can land. "No bug" is only acceptable for things like test infra updates or doc changes, anything which touches production code should have a bug in case we need to link it to regressions, use it in incident docs, etc.

Copy link
Contributor

@zzzeid zzzeid left a comment

Choose a reason for hiding this comment

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

lgtm but as @cgsheeh mentioned please file/add a bug to the title before landing this.

@shtrom shtrom changed the title scm: add SCMType enum to helpers (no bug) scm: add SCMType enum to helpers (bug 2019139) Feb 25, 2026
@lando-prod-mozilla
Copy link

Pull request closed by commit 9d7f841

lando-prod-mozilla bot pushed a commit that referenced this pull request Feb 25, 2026
This is a clean split of the SCMType work from Bug 1979247, to land separately.

Pull request: #945
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