Skip to content
New issue

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

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

Already on GitHub? # to your account

Add YMCA Website Services to Example Sites #2037

Merged
merged 2 commits into from
Jul 1, 2024
Merged

Conversation

froboy
Copy link
Contributor

@froboy froboy commented Jun 11, 2024

Thanks for all of your hard work!

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks @froboy, LGTM. See inline comment for suggested simplification.

Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
@froboy
Copy link
Contributor Author

froboy commented Jul 1, 2024

Pretty sure the failing tests are unrelated to this change...

@chalin
Copy link
Collaborator

chalin commented Jul 1, 2024

The Windows build failure has nothing to do with this PR. Merging.

@chalin chalin merged commit 309da20 into google:main Jul 1, 2024
10 of 11 checks passed
@chalin chalin added this to the 24Q3 milestone Aug 5, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants