Skip to content

WIP Bedrock support - #31

Draft
dingo-d wants to merge 15 commits into
developfrom
feature/bedrock-support
Draft

dingo-d wants to merge 15 commits into
developfrom
feature/bedrock-support

Conversation

@dingo-d

@dingo-d dingo-d commented Nov 10, 2023

Copy link
Copy Markdown
Owner

Description

Added support for Bedrock.

Still a WIP, as I'm still getting some errors when running the tests on Bedrock-type installs.

Also, need to update the changelog.

Closes #30

Checklist:

  • My code is tested.
  • My code follows the code style set by the project.
  • I've included documentation for the changes.

@dingo-d dingo-d self-assigned this Nov 10, 2023
@codecov

codecov Bot commented Nov 10, 2023

Copy link
Copy Markdown

Codecov Report

Merging #31 (dbbc031) into develop (b1e8df6) will increase coverage by 0.02%.
The diff coverage is 94.23%.

@@              Coverage Diff              @@
##             develop      #31      +/-   ##
=============================================
+ Coverage      88.70%   88.73%   +0.02%     
- Complexity        44       56      +12     
=============================================
  Files              1        1              
  Lines            186      222      +36     
=============================================
+ Hits             165      197      +32     
- Misses            21       25       +4     
Flag Coverage Δ
unittests 88.73% <94.23%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/Command/InitCommand.php 88.73% <94.23%> (+0.02%) ⬆️

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Notice about PHP 7 version was updated, as PHP 8 will be EOL soon.
Still a WIP. Need to fix tests, and test on bedrock projects for plugin and themes, and check if it will work in a regular projects like it did before.
Replace dirname with __FILE__ to __DIR__ and create a separate bootstrap files for bedrock, as the vendor file is usually at the project root, not where the theme or plugin is located.
Adding a required theme slug parameter is a bc change, as the command would fail if we just run it with theme option.
@dingo-d
dingo-d force-pushed the feature/bedrock-support branch from 9f7bbd5 to 17f6b84 Compare November 10, 2023 15:28
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.

1 participant