Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: improve logic for issue template closing and add tests #5905

Merged
merged 21 commits into from
Jan 16, 2025

Conversation

sofisl
Copy link
Contributor

@sofisl sofisl commented Dec 26, 2024

This PR was meant to only be to add better logic and testing for our issue automation since it was looking a little fragile, see #5838. I also will follow up for our split libraries, see googleapis/synthtool#2052

However, it also includes some changes that were incorrectly added to the library, namely #5914, #5381, #5922. The more permanent fixes will come in a new version of the generator, and cl/715614266.

The only relevant changes to review are in .github folder

@sofisl sofisl requested review from yoshi-approver and a team as code owners December 26, 2024 23:34
@sofisl sofisl added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 14, 2025
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 14, 2025
@sofisl sofisl closed this Jan 14, 2025
@sofisl sofisl reopened this Jan 14, 2025
Copy link
Contributor

@danieljbruce danieljbruce left a comment

Choose a reason for hiding this comment

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

Looks fine to me, but I don't know too much about this library.

@sofisl sofisl merged commit ecc4989 into main Jan 16, 2025
12 checks passed
@sofisl sofisl deleted the updateBugReport branch January 16, 2025 18:41
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