Skip to content

Conversation

@zwaxman
Copy link
Contributor

@zwaxman zwaxman commented Mar 17, 2025

Allow reporting isMergeCommit for the reportCommit command in our GH Action

Copy link
Contributor Author

zwaxman commented Mar 17, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@zwaxman zwaxman marked this pull request as ready for review March 17, 2025 15:51
@zwaxman zwaxman requested a review from a team as a code owner March 17, 2025 15:51
@zwaxman zwaxman requested review from shivghai and willhbang March 17, 2025 15:51
package.json Outdated
"@actions/core": "^1.10.1",
"@actions/github": "^6.0.0",
"ultralight-core": "0.2.5"
"ultralight-core": "canary"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remember to revert

@zwaxman zwaxman force-pushed the 03-17-ult-2813_merge_commit_in_reportcommit branch 4 times, most recently from dea6208 to edc42f0 Compare March 18, 2025 13:21
package.json Outdated
"@actions/core": "^1.10.1",
"@actions/github": "^6.0.0",
"ultralight-core": "0.2.5"
"ultralight-core": "canary"
Copy link
Contributor

Choose a reason for hiding this comment

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

revert?

Copy link
Contributor

@shivghai shivghai left a comment

Choose a reason for hiding this comment

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

approving pending revert

@zwaxman zwaxman force-pushed the 03-17-ult-2813_merge_commit_in_reportcommit branch from edc42f0 to 942fade Compare March 18, 2025 16:19
@zwaxman zwaxman force-pushed the 03-17-ult-2813_merge_commit_in_reportcommit branch from 942fade to 6ca9dc7 Compare March 18, 2025 16:24
@zwaxman zwaxman merged commit c8adc1d into main Mar 18, 2025
4 checks passed
@zwaxman zwaxman deleted the 03-17-ult-2813_merge_commit_in_reportcommit branch March 18, 2025 16:25
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