-
Notifications
You must be signed in to change notification settings - Fork 342
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
Decide on coding style and enforce mostly with PHPCS #299
Comments
Yes and no. I think PSR2 is a good base target to get to. I never got around to fully updating the existing code though. |
@markstory perhaps remove Sticker CI integration? it seems broken and no support. It's marked CI job as failed, example: #363 but can't retrieve any details why. the link from PR leads to https://stickler-ci.com/repositories/156-perftools-xhgui but that says "bummer, The requested address /repositories/156-perftools-xhgui was not found." The "login" on the page leads to https://stickler-ci.com/github/authorize, and redirects back to the same page, and even still shows "login" even I've logged in. I've contacted their support, and they have not even responded over several months. can you remove the integration? as I think I don't even have permission to remove the integration. |
@glensc Oddly enough I run stickler, and never got any support emails. I can look into what is not working with the login flow. There are build logs and results in the Github checks views though. |
@markstory ok, I can see output via GitHub actions, so I guess can leave as is then. The other link gave an error and I contacted their support, but the ☎️ is dead |
Less burden on reviewers, less guess work on the contributing person (ergo me). Also automation in supporting IDEs with automatic fixing/adapting as needed.
It looks like the current code looks mostly like PSR-2. Is that intentional?
(Extracted out of #16)
The text was updated successfully, but these errors were encountered: