Skip to content

Conversation

@sasajo
Copy link

@sasajo sasajo commented Sep 2, 2025

No description provided.

@sasajo sasajo requested a review from aarongoin September 16, 2025 21:29
@aarongoin aarongoin changed the base branch from main to develop September 16, 2025 21:45
Copy link
Member

@aarongoin aarongoin left a comment

Choose a reason for hiding this comment

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

Nice! Thanks Sasa!

Let's make sure we merge any changes into develop though. I've changed the base to be develop in your PR. Please look things over and make any adjustments if needed before merging.

@sasajo
Copy link
Author

sasajo commented Sep 17, 2025

Hi Aaron, it sims as can not be merged yet because there are some uncompleted checks : Maven Clean VerifyExpected — Waiting for status to be reported

@sasajo sasajo closed this Sep 17, 2025
@sasajo sasajo reopened this Sep 17, 2025
@aarongoin
Copy link
Member

@sasajo Okay I guess I had to manually approve the Maven Clean Verify workflow in order for it to run. That's odd. I'll look at why we need that--ideally it should just run. Anyway, you should be clear to merge now!

@aarongoin aarongoin merged commit a1c57eb into polyapi:develop Sep 19, 2025
1 check passed
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