Skip to content

comments handled from upstream#4

Merged
ankitml merged 1 commit intomainfrom
ankitml/upstream-comments
Jan 23, 2026
Merged

comments handled from upstream#4
ankitml merged 1 commit intomainfrom
ankitml/upstream-comments

Conversation

@ankitml
Copy link
Collaborator

@ankitml ankitml commented Jan 23, 2026

Add git ref validation and use spawnSync for safer command execution

Copy link
Member

@philippemnoel philippemnoel left a comment

Choose a reason for hiding this comment

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

let's squash all of this into a single commit?

@ankitml
Copy link
Collaborator Author

ankitml commented Jan 23, 2026

@philippemnoel yes. more is coming from the comments. I want to test small changes in our repo so we can be sure of things.

@ankitml ankitml merged commit 549f163 into main Jan 23, 2026
1 check passed
@ankitml ankitml deleted the ankitml/upstream-comments branch January 23, 2026 22:21
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