Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update plugin structure#18

Merged
stefanorosanelli merged 12 commits into
masterfrom
chore/update-plugin-structure
Mar 28, 2022
Merged

Update plugin structure#18
stefanorosanelli merged 12 commits into
masterfrom
chore/update-plugin-structure

Conversation

@fquffio

@fquffio fquffio commented Mar 28, 2022

Copy link
Copy Markdown
Member

This PR updates plugin structure using the Plugin class, introduced a few CakePHP versions ago.

It also updates some code-quality tools configurations, and raises PHP STAN level to 9 (max level).

@codecov

codecov Bot commented Mar 28, 2022

Copy link
Copy Markdown

Codecov Report

Merging #18 (dcc86b3) into master (a559769) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master       #18   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity         9        13    +4     
===========================================
  Files              3         4    +1     
  Lines             33        43   +10     
===========================================
+ Hits              33        43   +10     
Impacted Files Coverage Δ
src/Middleware/HtmlMiddleware.php 100.00% <ø> (ø)
src/Plugin.php 100.00% <100.00%> (ø)
src/Shell/Task/ResourcesMigrationTask.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a559769...dcc86b3. Read the comment docs.

Comment thread src/Plugin.php Outdated
Comment thread tests/TestCase/PluginTest.php Outdated
Comment thread tests/TestCase/PluginTest.php Outdated
@stefanorosanelli stefanorosanelli merged commit a603291 into master Mar 28, 2022
@stefanorosanelli stefanorosanelli deleted the chore/update-plugin-structure branch March 28, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants