Skip to content

Conversation

@lillianberryfly
Copy link
Contributor

No description provided.

@lillianberryfly lillianberryfly marked this pull request as draft November 12, 2025 14:59
@lillianberryfly lillianberryfly marked this pull request as ready for review November 13, 2025 09:53
Copy link
Member

@jphenow jphenow left a comment

Choose a reason for hiding this comment

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

Hard to zero in on anything specific but the bits I scanned make sense. Let me know if there's anything specific I should focus on?

There was at least one spot where we do more inspecting of "personal" - should just make sure launch works on personal as expected still etc. if you haven't

CI failures seem related or no?

@lillianberryfly
Copy link
Contributor Author

There was at least one spot where we do more inspecting of "personal" - should just make sure launch works on personal as expected still etc. if you haven't

I've been testing on personal org, it works yep

CI failures seem related or no?

it's also failing on main 🤷 have been trying to figure out why but no luck

@lillianberryfly lillianberryfly merged commit 0493f22 into master Nov 14, 2025
21 of 30 checks passed
@lillianberryfly lillianberryfly deleted the less-graphql-launch-deploy branch November 14, 2025 08:46
@sentry
Copy link

sentry bot commented Nov 14, 2025

Issues attributed to commits in this pull request

This pull request was merged and Sentry observed the following issues:

swilson-fly added a commit that referenced this pull request Nov 14, 2025
swilson-fly added a commit that referenced this pull request Nov 14, 2025
rianmcguirefly added a commit that referenced this pull request Nov 17, 2025
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