github.com/joomla/coding-standards commits | Ecosyste.ms: OpenCollective

PSR-12

0becb28c7ae1e0b19e6ff41725618c75f2cb0b47 authored over 2 years ago

Merge pull request #273 from PhilETaylor/patch-5

Propose [] over array() going forward from Joomla 4

87bec25c015a8f269a178fb822ffcb2674a344f6 authored about 3 years ago

apply suggestion

22ee6e164a04857abbcaf727f64129cd2d87cae0 authored over 3 years ago

The use of the `@` for Error Surpression should be avoided (#274)

f14a3d1aefaa586c37ebd23c777ec6582b354f98 authored over 3 years ago

Propose [] over array() going forward from Joomla 4

1352428ab4e409c066b30853ac6ce6a49a4ae7c4 authored over 3 years ago

remove "You should update the contribution log in CREDITS.php." (#262)

2bc5e324c218f72c8bdc05563cecf78392202d74 authored over 4 years ago

s/brace/bracket - correct terminology (#265)

db997774b88a7cb7b55b8ed297e284f02f6fef4e authored over 4 years ago

remove hyphen that's incorrectly placed (#266)

535dcc2920a0f119d2872a2aa2efafbd73d6ad72 authored over 4 years ago

Merge pull request #268 from Hackwar/patch-3

Fixing list of magic methods for naming convention

d6a62eab09e342be51be2b30030b61071e26c8ca authored over 4 years ago

Update ValidFunctionNameSniff.php

change to old array syntax

29c960dc85424af29d2fc0582f77cd6053c7c9f0 authored over 4 years ago

Fixing list of magic methods for naming convention

4d573b6feb9cc0adba2b9db8527f50f30472dbd0 authored over 4 years ago

Merge pull request #258 from jrfnl/feature/fix-php-7.4-compat

Fix compatibility with PHP 7.4

1ceeaf4bfe6b40c331a0976b93b38df80b8b386a authored almost 5 years ago

Fix compatibility with PHP 7.4

Array and string offset acces using curly braces is deprecated since PHP 7.4.

Ref: https://www....

8bb22db217fb02fe01b625b8c9613815b167ab74 authored almost 5 years ago

Don't allow package tags in namespaced files (#255)

c6970aa6c17d422cee9bc18fab0fd7fbb204e4ef authored about 5 years ago

Merge pull request #257 from perry-contribs/master

typo fix - README.md, fixes #256

8d79cb2b751e6c9deb3f710423d92eebfa42d2a2 authored over 5 years ago

typo fix - README.md

compatable -> compatible
selectivly -> selectively
dependancy -> dependency
seperated -> sepa...

3118b0a4603a3f96abf1c991b6916d60c1ff5e22 authored over 5 years ago

Minor edits

687c05031476c09b45b97e3f7ab524254092c5bd authored over 5 years ago

Merge pull request #253 from Sandra97/patch-1

Double blank space

cdc0a0e149ef88e15985a4ef3986bcdf94335786 authored over 5 years ago

Double blank space

There was some double blank space, maybe it can explain the unwanted <br> on the dev site?

76b4e5e42228eb769fd8492d04bcd49c63510ccb authored over 5 years ago

Fix link format

01b3895c240c3e3a4ec9ef66933bdf1812cfeb19 authored over 5 years ago

Add separate Joomla 4 javascript page (#252)

5e30f717a0e33b6ae7f9cd37b79d40096ef7974c authored over 5 years ago

Add test case for issue found in the CMS with closures (#250)

bcd5ae7cdc35549706a3d5ace3d67a0c3e0fd052 authored over 5 years ago

Merge pull request #251 from Hackwar/patch-2

Adding testrunners for PHP7.3/.4

ce73dc37c52e1e5b8df723a1647abb214652c8dc authored over 5 years ago

Adding testrunners for PHP7.3/.4

83c5e43f217e987c7e4282babdb2862c44807189 authored over 5 years ago

Add note on Exclamation mark `!` logical operator (#220)

Reference https://github.com/joomla/coding-standards/issues/113

8a70e4dd303b6b89165986a3cecccde094a71c30 authored almost 6 years ago

Fix Implicit true comparison (#244)

best practice to always avoid Implicit true comparisons (also mandated by PHPCS when contributin...

148bb426cb973c7e448d10be254dfd2adbbaa042 authored almost 6 years ago

Revert 9b073ca9e69a63e6309dfe6cb941a315bdea2030 as GitHub doesn't support it

f49b68686b319327687f30a11eefad54a0bfe513 authored almost 6 years ago

Set an ID for the General heading

9b073ca9e69a63e6309dfe6cb941a315bdea2030 authored almost 6 years ago

Add codestyle for language files (#241)

c2bed76b0a90d1986ef11b6f12d33a21facf017e authored about 6 years ago

Merge pull request #243 from photodude/patch-4

Fixes outdated composer command in instructions

6e003c6950e52ea0531ec42857c13df43c802a1f authored over 6 years ago

Fixes outdated composer command in instructions

applies composer command fix in #233 to master branch

a57efcaa7f2e3765b19039eecba5526c7a1a0ad0 authored over 6 years ago

[WIP] Update ruleset to represent current CMS state (#214)

* Update ruleset to represent current CMS state

* Fix build exclude pattern

- the build ex...

42206ee46740d3e24a58f3c10af695b3a7a490d7 authored over 6 years ago

Add test for second multiple parenthesis

caa06aa832d1357bb2cefda521604098b00b3186 authored over 6 years ago

Merge pull request #242 from wilsonge/fix/brackets

Fix issues with closures. Fixes #230 and #231

01b9a2d6e3f3cfedd6f28d262b11cfbf811ba3ab authored over 6 years ago

Line numbers of errors

8f7db67ce2f97f00e825fc05e46c5794aa18ee80 authored over 6 years ago

Add some more errors

c2b95a386cba3bd9750188f72a1de69a84cd9b46 authored over 6 years ago

Code style

720613cbe0a9621bc5dd7972b7a39a0f16d765e9 authored over 6 years ago

Fix issues with closures. Fixes #230 and #231

4c1503f1ae304f4a83f012751730b1e03ecfdbc2 authored over 6 years ago

Add .idea to .gitignore

bef9d1acc0f33401592083e21b60ae14c21dd1a6 authored over 6 years ago

Manual formatting

fc420f31e38d6af8ff1c895622893b129d6ebf86 authored over 6 years ago

Optimize links for display on dev site

9341c0824ab386a25e1ea5d2d8e19c1cb8d16a36 authored over 6 years ago

Merge pull request #239 from roland-d/namespace-use-statement

Define order of namespace imports

806064525493bd4150421f2da96b5b3fead5e96f authored over 6 years ago

Slight tweaks.

a91216d2e0f83daac9f01579a5cd9bea9e29aa3b authored over 6 years ago

Updated text

Signed-off-by: Roland Dalmulder <[email protected]>

1e6f4d748aa42fc8bf96bf48dff0d1f3eb6a10dd authored over 6 years ago

Define order of namespace imports

Signed-off-by: Roland Dalmulder <[email protected]>

4d5cefaa96d1ec9b60d2c0461d236d4ab0d1b5be authored over 6 years ago

Merge pull request #238 from roland-d/quotename-not-qn

We should use quoteName and not qn

f8cb1ab5bbc149ac1dce03a5856bbf75453ac1b1 authored over 6 years ago

We should use quoteName and not qn

Signed-off-by: Roland Dalmulder <[email protected]>

ee2471ae4bec37a429f052cdbb8294460390fe38 authored over 6 years ago

Merge pull request #232 from photodude/patch-4

Fix failing CS

1a93124564f2be7ca0a301b755249c65fe111324 authored almost 7 years ago

Multi-line IF statement not indented correctly

e9a8ee8915c390e3316cd135aaf4167d737a92d1 authored almost 7 years ago

Edit license identifier

4a9e350d7c55049df69bf7fa1d7b0be929537946 authored almost 7 years ago

Merge pull request #229 from photodude/patch-2

Fix return type comments

dc2f8b2f7b767fafd76af8332ec13d23be0c5eb9 authored almost 7 years ago

repeat code style fixes

cd0893e9fff5ca9d1c2306e230e19fa0811cf96c authored almost 7 years ago

code style fixes

07061725c689edbde5dc1c1d290ea4ddf804de3e authored almost 7 years ago

correct test alignment and bracket placement

fe78cf3d65381b99523b1ba60e042769eaa04a21 authored almost 7 years ago

update fixed tests with closure and comment

b9dec0ba5f5f528c962770588f0137352b81bd62 authored almost 7 years ago

New tests for closures and comments

3bb839fc867c652a4972bbe9a674223ea0b7edcf authored almost 7 years ago

Support both a return type and a description

adopt some fixes from Squiz.Commenting.FunctionComment changes in the 3.x branch

5d0b8ccf974505de0b0b7a008dea0cad22d1ad3d authored almost 7 years ago

Merge pull request #223 from gruz/patch-1

Eratta

6f98a125efb93abdf181fd745bf87799076fb332 authored almost 7 years ago

Eratta

4cfec2b2a8a9ac018a8097401fd841697b37973c authored almost 7 years ago

Merge pull request #218 from photodude/patch-6

Fix closure indent issue and add tests

b09a00e46f83e8b2d21efaa95acfbd67fd84f9bd authored about 7 years ago

Add new expected errors for Anonymous classes test

52c6c48f1b6c5f552cef9af0fd0dc8c506ae7a91 authored about 7 years ago

Add Anonymous classes test

25becd4afd21520b01bb63da30804762ed27b845 authored about 7 years ago

add Anonymous classes test

9187a7ec5f17c285e166c5e67cfc21d76ef4c3ea authored about 7 years ago

Fix indent within Anonymous classes and functions

21308113febaa6debdc481a9bb7d35668d394963 authored about 7 years ago

Add expected errors on lines 74 and 78

1c8984c4bcbcc9fc96ba8d73bcecdc5c913c6c57 authored about 7 years ago

modify closure test

b61ceb93c6830c27d18578c5eecd732da61aadab authored about 7 years ago

Add closure test fixed

59f55dcfb2f87848080d4a636518f261ff0d8727 authored about 7 years ago

rename function with closure to foo

22c9fb8691f816da27f4958514b0aa0303bd9053 authored about 7 years ago

Add closure indent test

4e71ed8f99aa4a1f37e6a38ae6fd0f378541c6ce authored about 7 years ago

Merge pull request #204 from joomla/optional-package

Make file package tag optional

7aee4b92c7fac5f1099483d57b0d291618755d85 authored about 7 years ago

Merge pull request #213 from photodude/patch-6

Note: package tag is optional in namespaced code

403625fafb3afd4406ca0b12c486aa1fec7a3496 authored about 7 years ago

Add stricter information on how to use @param (#205)

8048f7d09e9abdfba0c5dd0beb94f5c0b57a5619 authored about 7 years ago

Note: package tag is optional in namespaced code

3de11b0c383b81861b7f1350d46fabd136a418c0 authored about 7 years ago

Merge pull request #203 from photodude/patch-4

We don't need to include PHPCS in composer.json instructions

4a10190d0b4dbe3d3002548d36aff5fe19df6988 authored over 7 years ago

Merge pull request #209 from photodude/patch-7

Add PHP 7.2 as allowed failure

e7956b53d918e8e469e1e506829ecc2495e3aaab authored over 7 years ago

Add php 7.2 as allowed failure

7c8272362b6b99cb5cc89a90b326f00ebf607b69 authored over 7 years ago

Make file package tag optional

baf5bce2ec2925908fec73cef1d1a991e82ec1a3 authored over 7 years ago

We don't need to include PHPCS in composer.json

PHPCS is included because of the coding standards requirements
also note the alpha install wit...

86de4d8e51f8f0cac66db00b06c8f53a3e67fb2e authored over 7 years ago

Merge pull request #200 from photodude/patch-4

Update note for PHPCS 3.x series

6967330a855fe83d24270f01d819f4d32180a94e authored over 7 years ago

Update note for PHPCS 3.x series

1aa1d4abc5dfb24643bc91638fd1afa25ff75684 authored over 7 years ago

Remove HHVM and add nightly (#196)

3eeab4d1d1a4d87130f975cde8b2a7b5a6a36340 authored over 7 years ago

clarify function space rules for function definition and closures (#194)

* no spaces between the function definition name and the opening parenthesis

* add example on...

8c08db51a45b2e13377fdf601c5748ec37e0f293 authored over 7 years ago

Add badges, minor README updates

46a9fa4d4db18b90cc9196349e9f7a8e8dc7873a authored over 7 years ago

This is 2.x dev

1b25faeae877721e902661f0427ac3b659d1663e authored over 7 years ago

Merge pull request #189 from brianteeman/patch-1

modernise

47594ec4e7dd5f1c5a49b65ccb34d8b4dc2c0b81 authored over 7 years ago

Merge pull request #188 from yvesh/patch-3

Add composer stability note to the README.md

f4116d12fafd5e1ab40a48224c0c85cd5824e0aa authored over 7 years ago

modernise

remove ref to en-au and add references to the style guide

a07ed7773ead3df692b3d3abff85f55b53da8627 authored over 7 years ago

Add doku link for setting up codesniffer (#185)

* Update analysis.md

3ed506700140e8079c5d8b033d71a8a1f909e80f authored over 7 years ago

Add composer stability note to the README.md

To install the current joomla/coding-standards 2.0.0 version over composer, you need to set mini...

807a7d45b07bb995bf4ba7e1baf868df37e23be0 authored over 7 years ago

Merge pull request #186 from frenck/feature/composer-installer-support

Changed Composer package type to phpcodesniffer-standard

6ddccdc820a15e9c3219b2e2f788eced6b5ece54 authored over 7 years ago

Merge pull request #187 from photodude/patch-3

fix CS tabs use

e2159402ed31da6e200b13ba66694cc11379804a authored over 7 years ago

fix CS tabs use

8f638288f31c962bed17e6e87bee5ec8e99df7ea authored over 7 years ago

Merge pull request #183 from photodude/patch-3

add notes about using phpcs --config-set

c3817097f2ba42de5b5033537b8614450ecd4c29 authored over 7 years ago

Changed Composer package type to phpcodesniffer-standard

fee4eac0a5aadca6b48dbe422ca295e589b6070b authored over 7 years ago

add more specifics for using phpcs --config-set

b7526dd5ab5e34f00e5be005c626b305863ed9ba authored over 7 years ago

add notes about using phpcs --config-set

reference
https://github.com/squizlabs/PHP_CodeSniffer/wiki/Configuration-Options#setting-the-i...

e317262a6b9a0721f316078c1f7cb25cd6637516 authored over 7 years ago

Merge pull request #180 from 810/patch874634

Missing param shortArraySyntax #179

8d4dda7c19a98658ea83ee3806858553a6057a7c authored over 7 years ago

Update code

2726948b54efb79317fa3b7c16111beffc9e2be5 authored over 7 years ago

Missing param shortArraySyntax #179

Or should it false at default?

dc51f9af89e435e68d88e8b826508858eb957ce1 authored over 7 years ago

Merge pull request #173 from uglyeoin/patch-1

made end link a link

a0a08ea41cd55439270929ca4f78cb16def730bc authored almost 8 years ago