Skip to content

cs fixer v3.16 moves method braces to a new line #11

@ChristophWurst

Description

@ChristophWurst

Ref nextcloud/mail#8334

 472) tests/HordeTranslationHandlerTest.php
      ---------- begin diff ----------
--- /home/runner/work/mail/mail/tests/HordeTranslationHandlerTest.php
+++ /home/runner/work/mail/mail/tests/HordeTranslationHandlerTest.php
@@ -23,16 +23,19 @@
 use ChristophWurst\Nextcloud\Testing\TestCase;
 use OCA\Mail\HordeTranslationHandler;
 
-class HordeTranslationHandlerTest extends TestCase {
+class HordeTranslationHandlerTest extends TestCase
+{
 	private $handler;
 
-	protected function setUp(): void {
+	protected function setUp(): void
+	{
 		parent::setUp();
 
 		$this->handler = new HordeTranslationHandler();
 	}
 
-	public function testT() {
+	public function testT()
+	{
 		$message = 'Hello';
 
 		$expected = $message;
@@ -41,7 +44,8 @@
 		$this->assertEquals($expected, $actual);
 	}
 
-	public function singularPluralDataProvider() {
+	public function singularPluralDataProvider()
+	{
 		return [
 			[0],
 			[1],
@@ -52,7 +56,8 @@
 	/**
 	 * @dataProvider singularPluralDataProvider
 	 */
-	public function testNgettext($number) {
+	public function testNgettext($number)
+	{
 		$singular = 'mail';
 		$plural = 'mails';
 

      ----------- end diff -----------

This shared config still uses the deprecated braces option https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/blob/master/doc/rules/basic/braces.rst

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions