Ticket #3040 - Updated Stripe API

plugins/stripe/.coveralls.yml → plugins/stripe/.coveralls.github-actions.yml

@@ -1,4 +1,3 @@
-service_name: travis-ci
-src_dir: .
+service_name: github-actions
 coverage_clover: clover.xml
 json_path: coveralls-upload.json

plugins/stripe/.editorconfig

@@ -0,0 +1,20 @@
+; https://editorconfig.org/
+
+root = true
+
+[*]
+indent_style = space
+indent_size = 4
+insert_final_newline = true
+trim_trailing_whitespace = true
+end_of_line = lf
+charset = utf-8
+
+[*.{json,yml}]
+indent_size = 2
+
+[*.neon]
+indent_style = tab
+
+[Makefile]
+indent_style = tab

plugins/stripe/.gitignore

@@ -1,3 +1,6 @@
+# Ignore build files
+build/*
+
 # Mac OS X dumps these all over the place.
 .DS_Store
 
@@ -15,3 +18,14 @@ clover.xml
 
 # Ignore IDE's configuration files
 .idea
+
+# Ignore PHP CS Fixer local config and cache
+.php_cs
+.php_cs.cache
+
+# Ignore PHPStan local config
+.phpstan.neon
+
+# Ignore phpDocumentor's local config and artifacts
+.phpdoc/*
+phpdoc.xml

plugins/stripe/.php_cs.dist

@@ -0,0 +1,38 @@
+<?php
+
+return PhpCsFixer\Config::create()
+    ->setRiskyAllowed(true)
+    ->setRules([
+        // Rulesets
+        '@PSR2' => true,
+        '@PhpCsFixer' => true,
+        '@PhpCsFixer:risky' => true,
+        '@PHP56Migration:risky' => true,
+        '@PHPUnit57Migration:risky' => true,
+
+        // Additional rules
+        'fopen_flags' => true,
+        'linebreak_after_opening_tag' => true,
+        'native_function_invocation' => true,
+
+        // --- Diffs from @PhpCsFixer / @PhpCsFixer:risky ---
+
+        // This is just prettier / easier to read.
+        'concat_space' => ['spacing' => 'one'],
+
+        // This causes strange ordering with codegen'd classes. We might be
+        // able to enable this if we update codegen to output class elements
+        // in the correct order.
+        'ordered_class_elements' => false,
+
+        // Keep this disabled to avoid unnecessary diffs in PHPDoc comments of
+        // codegen'd classes.
+        'phpdoc_align' => false,
+
+        // This is a "risky" rule that causes a bug in our codebase.
+        // Specifically, in `StripeObject.updateAttributes` we construct new
+        // `StripeObject`s for metadata. We can't use `self` there because it
+        // needs to be a raw `StripeObject`.
+        'self_accessor' => false,
+    ])
+;