Skip to content

fix(services/ghac)!: Remove enable_create_simulation support for ghac#3423

Merged
Xuanwo merged 3 commits intomainfrom
remove-ghac-create
Oct 30, 2023
Merged

fix(services/ghac)!: Remove enable_create_simulation support for ghac#3423
Xuanwo merged 3 commits intomainfrom
remove-ghac-create

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Oct 30, 2023

No description provided.

Signed-off-by: Xuanwo <github@xuanwo.io>
@github-actions github-actions Bot added the releases-note/fix The PR fixes a bug or has a title that begins with "fix" label Oct 30, 2023
@Xuanwo
Copy link
Copy Markdown
Member Author

Xuanwo commented Oct 30, 2023

Just to be sure, @huonw, are you relying on this feature?

Comment thread core/src/docs/upgrade.md Outdated
Co-authored-by: Suyan <suyanhanx@gmail.com>
@Xuanwo Xuanwo marked this pull request as ready for review October 30, 2023 09:32
@Xuanwo Xuanwo requested a review from PsiACE as a code owner October 30, 2023 09:32
@Xuanwo
Copy link
Copy Markdown
Member Author

Xuanwo commented Oct 30, 2023

cc @suyanhanx, PTAL, thanks!

Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

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

Thanks

@Xuanwo Xuanwo merged commit d77c361 into main Oct 30, 2023
@Xuanwo Xuanwo deleted the remove-ghac-create branch October 30, 2023 09:56
@huonw
Copy link
Copy Markdown
Contributor

huonw commented Oct 30, 2023

No, Pants doesn't use this. It handles the empty file in a different way. Thanks for checking!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/fix The PR fixes a bug or has a title that begins with "fix"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants