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

Add Laravel #181

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Add Laravel #181

merged 3 commits into from
Oct 8, 2024

Conversation

christophrumpel
Copy link
Contributor

No description provided.

@chadwhitacre
Copy link
Contributor

Woo-hoo!

@vladh Michael and I have been working with Laravel to get them onboarded. Lmk how it looks to you, I'm good to merge if you are.

@vladh
Copy link
Member

vladh commented Oct 8, 2024

Amazing, so nice to have you on board! 🎉

I’m seeing some broken Unicode characters in the description, is that just me? Looks like em dashes.

@chadwhitacre
Copy link
Contributor

Where are you seeing them break, @vladh? I believe they are em dashes, yes.

@chadwhitacre
Copy link
Contributor

They look fine here. At this point I would say let's prioritize landing this and then follow up with cleanups. We're on the HN homepage and we want Laravel to be included in the marketing cycle today.

image

@vladh vladh merged commit 7ffb5ab into opensourcepledge:main Oct 8, 2024
1 check passed
@vladh
Copy link
Member

vladh commented Oct 8, 2024

Must be on my end, let’s get it merged!

@chadwhitacre
Copy link
Contributor

Yeah also we have them on the homepage already with #157 lol

@chadwhitacre
Copy link
Contributor

Lgtm https://opensourcepledge.com/members/laravel/

image

Marked as innovator in 2722b6e.

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