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.

Sign up for GitHub

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

Use PSR-12 instead of PSR-2 #137

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
+8 −8
Open

Use PSR-12 instead of PSR-2 #137

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
647f29e
Use PSR-12 instead since PSR-2 is now deprecated
gmponos Aug 24, 2019
ef02cd0
Use PSR-12 instead of PSR-2
gmponos Aug 24, 2019
1fbf7e1
User codesniffer greater than 3.3 for PSR-12
gmponos Aug 24, 2019
f0fcefa
updated codesniffer version
gmponos Oct 9, 2019
ae1deaa
Merge branch 'master' into patch-1
gmponos Dec 4, 2019
34f486c
Merge remote-tracking branch 'origin/master' into patch-1
gmponos Jun 19, 2020
67efaac
Updated URL
gmponos Jun 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Updated URL
@gmponos
gmponos committed Jun 19, 2020
commit 67efaac7442d7dfb3cd0e6efbbb0192f7d190d61
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
View file
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@ We accept contributions via Pull Requests on [Github](https://github.com/:vendor

## Pull Requests

- **[PSR-12 Coding Standard](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-12-extended-coding-style-guide.md)** - Check the code style with ``$ composer check-style`` and fix it with ``$ composer fix-style``.
- **[PSR-12 Coding Standard](https://www.php-fig.org/psr/psr-12/)** - Check the code style with ``$ composer check-style`` and fix it with ``$ composer fix-style``.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Due to the changes happening to the dev community (master/main branches) I updated this to point to the website instead.

(friendly ping again)


- **Add tests!** - Your patch won't be accepted if it doesn't have tests.