Feature #38534

Activate CGL rule "Squiz.WhiteSpace.SemicolonSpacing" in ci.typo3.org-gerrit-slave

Added by Roland Waldner 12 months ago. Updated 11 months ago.

Status:Resolved Start date:2012-07-02
Priority:Should have Due date:
Assignee:Andy Grunwald % Done:

0%
Category:-
Target version:-
Votes: 0

Description

Some days ago i noticed, that metrics.typo3.org showed 2 new CGL violations against "SemicolonSpacingIncorrect". This was resolved yesterday and the commit was merged: metrics.typo3.org does not show any remaining CGL against this rule any longer.

History

#1
Updated by Roland Waldner 12 months ago

the sniff seems to be called "Squiz.WhiteSpace.SemicolonSpacing".

#3
Updated by Roland Waldner 12 months ago

the current nightly build job on ci.typo3.org does not show any violatins against this rule neither:

https://ci.typo3.org/view/TYPO3/view/Core/job/typo3-v4-core-nightly/566/checkstyleResult/?

#4
Updated by Andy Grunwald 12 months ago

  • Status changed from New to Under Review
  • Assignee set to Andy Grunwald

I will take care of this issue.

Please be so nice to only create tickets about activation of sniffs and not post this messages at the mailing list, because if everyone post this stuff at the mailing list, we don`t have the overview of sniffs to activate, ok?

#5
Updated by Roland Waldner 12 months ago

hi,

ok. thanks for updating the wiki so this most likely will not happen any more.

kind regards.

roland

#6
Updated by Andy Grunwald 12 months ago

Copy from the mailinglist:

Hey Roland,

we will check your sniff and will activated in the next "activation run".
I think this will be around the weekend.
We don`t want to activate many sniffs in one run, because if we got trouble with the builds it is much more easier to determine the error.

I hope you undertand this.
Thanks!

Andy

#7
Updated by Roland Waldner 12 months ago

perfect. thanks.

#8
Updated by Andy Grunwald 12 months ago

  • Subject changed from Activate "SemicolonSpacingIncorrect" in ci.typo3.org-gerrit-slave on ci.typo3.org to Activate CGL rule "Squiz.WhiteSpace.SemicolonSpacing" in ci.typo3.org-gerrit-slave

#10
Updated by Andy Grunwald 11 months ago

  • Status changed from Under Review to Resolved

Sniff is active, now.

Also available in: Atom PDF