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 lighthouse to implementations #60

Merged
merged 12 commits into from
Mar 31, 2023
Merged

Add lighthouse to implementations #60

merged 12 commits into from
Mar 31, 2023

Conversation

spawnia
Copy link
Member

@spawnia spawnia commented Mar 21, 2023

This merge request adds https://lighthouse-php.com as an implementation.

.github/workflows/audits.yml Outdated Show resolved Hide resolved
.github/workflows/audits.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
.github/workflows/audits.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
.github/workflows/audits.yml Outdated Show resolved Hide resolved
@enisdenjo
Copy link
Member

This PR is good, but let me first take care of #60 (comment) because Lighthouse is compliant and it deserves to be in the compliant servers table.

@spawnia
Copy link
Member Author

spawnia commented Mar 21, 2023

I have not had too much trouble with adding this implementation, but it might be valuable to have a guide for it. Would you welcome if I added one in a separate pull request? I would use CONTRIBUTING.md for it.

@enisdenjo
Copy link
Member

Would you welcome if I added one in a separate pull request? I would use CONTRIBUTING.md for it.

Of course, more than welcome! Thank you in advance!

Copy link
Member

@enisdenjo enisdenjo left a comment

Choose a reason for hiding this comment

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

Please rebase this PR on main because #29 should be fixed now.

@spawnia
Copy link
Member Author

spawnia commented Mar 30, 2023

Thanks @enisdenjo, I am happy to see that Lighthouse now passes the audit.

enisdenjo added a commit that referenced this pull request Mar 31, 2023
Seems like when using `pull_request_target` trigger wont allow changes to the workflow file itself.
@enisdenjo enisdenjo merged commit 365f9f0 into graphql:main Mar 31, 2023
@spawnia spawnia deleted the lighthouse branch March 31, 2023 13:13
@enisdenjo
Copy link
Member

🎉 This PR is included in version 1.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@enisdenjo enisdenjo added the released Has been released and published label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Has been released and published
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants