Skip to content

Closes #20 : Group rules into few categories by priority#23

Open
PauliusMacernis wants to merge 1 commit into
masterfrom
feature/group_rules_into_categories
Open

Closes #20 : Group rules into few categories by priority#23
PauliusMacernis wants to merge 1 commit into
masterfrom
feature/group_rules_into_categories

Conversation

@PauliusMacernis

Copy link
Copy Markdown
Member

Group rules into few categories by priority, disable rules that contradicts standards or falls into discussable "other" category (these may come back after discussed), change psr2 to psr12 (psr12 includes compatible part of psr2, psr2 is deprecated as of 2019-08-10).

…es that contradicts standards or falls into discussable "other" category (these may come back after discussed), change psr2 to psr12 (psr12 includes compatible part of psr2, psr2 is deprecated as of 2019-08-10).
@jbieliauskas

Copy link
Copy Markdown
Collaborator

Check out #25 on Closes.

@jbieliauskas jbieliauskas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please add new lines at the end of files.

@PauliusMacernis

PauliusMacernis commented Nov 12, 2019

Copy link
Copy Markdown
Member Author

Please add new lines at the end of files.

PSR2:

All PHP files MUST end with a single blank line.

As of 2019-08-10 PSR-2 has been marked as deprecated. PSR-12 is now recommended as an alternative.

PSR12:

This specification extends, expands and replaces PSR-2, ...

All PHP files MUST end with a non-blank line, terminated with a single LF.

Probably needs to be investigated more?

Update:
php-fig/fig-standards#1173

@jbieliauskas

Copy link
Copy Markdown
Collaborator

You're saying that it's not a php file, so doesn't have to include a LF at the end?

The line feeds are a UNIX thing. A line isn't considered a line if it isn't followed by a LF. Most standard tools require this, for example, wc -l standards/akeneo.yaml yields 13 instead of 14. It's a general practice.

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.

2 participants