Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[!!!][FEATURE] Introduce project:create console command #86

Merged
merged 13 commits into from
Jul 7, 2023

Conversation

eliashaeussler
Copy link
Member

@eliashaeussler eliashaeussler commented Jan 20, 2023

With this PR, the project creation logic is extracted into a separate console command. The command is exposed to the Composer application, allowing to execute it through Composer as composer project:create. In order to make this change possible, the package is converted to a Composer plugin.

This has no impact on the previous single use case – composer create-project. Instead, it provides a better abstraction of the project creation logic and allows to execute the project creation with having the project builder as (dev) dependency.

This also paves the way for an implementation of #85.

ToDo:

  • Documentation
  • Tests

@eliashaeussler eliashaeussler added enhancement New feature or request breaking This introduces a breaking change labels Jan 20, 2023
@eliashaeussler eliashaeussler self-assigned this Jan 20, 2023
@codeclimate
Copy link

codeclimate bot commented Jan 20, 2023

Code Climate has analyzed commit a2ca0ca and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.7% (1.4% change).

View more on Code Climate.

@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #86 (31c02a6) into develop (4c0f9e3) will increase coverage by 1.54%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop      #86      +/-   ##
=============================================
+ Coverage      96.12%   97.66%   +1.54%     
- Complexity       656      692      +36     
=============================================
  Files             98      101       +3     
  Lines           2089     2186      +97     
=============================================
+ Hits            2008     2135     +127     
+ Misses            81       51      -30     
Impacted Files Coverage Δ
src/Builder/Generator/Step/CleanUpStep.php 100.00% <ø> (+2.43%) ⬆️
...uilder/Generator/Step/MirrorProcessedFilesStep.php 100.00% <ø> (ø)
src/Builder/Config/ConfigFactory.php 100.00% <100.00%> (ø)
src/Builder/Config/ConfigReader.php 100.00% <100.00%> (ø)
...ilder/Generator/Step/GenerateBuildArtifactStep.php 100.00% <100.00%> (ø)
src/Console/Command/CreateProjectCommand.php 98.78% <100.00%> (ø)
src/Console/Command/ProxyCommand.php 100.00% <100.00%> (ø)
src/Console/IO/AccessibleConsoleIO.php 100.00% <100.00%> (ø)
src/Exception/FilesystemFailureException.php 100.00% <100.00%> (ø)
src/Exception/InvalidConfigurationException.php 100.00% <100.00%> (+18.18%) ⬆️
... and 4 more

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@eliashaeussler eliashaeussler force-pushed the feature/console-command branch 3 times, most recently from 535b0f7 to 54636b7 Compare January 23, 2023 14:34
@eliashaeussler eliashaeussler force-pushed the feature/console-command branch 2 times, most recently from 2cd73a7 to e452954 Compare February 1, 2023 16:16
@eliashaeussler eliashaeussler force-pushed the feature/console-command branch 3 times, most recently from def22f7 to fccfb9d Compare February 1, 2023 18:31
@eliashaeussler eliashaeussler marked this pull request as ready for review March 19, 2023 19:03
@eliashaeussler
Copy link
Member Author

@mteu This PR is now ready for review.

@mteu
Copy link
Collaborator

mteu commented Mar 19, 2023

Thanks, I'll see to it.

@eliashaeussler eliashaeussler modified the milestones: v2.0.0, v3.0.0 Mar 19, 2023
@eliashaeussler
Copy link
Member Author

@mteu ping

@mteu mteu changed the base branch from develop to 3.x July 7, 2023 09:49
Copy link
Collaborator

@mteu mteu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💛

@eliashaeussler eliashaeussler merged commit e9d42bc into 3.x Jul 7, 2023
23 checks passed
@eliashaeussler eliashaeussler deleted the feature/console-command branch July 7, 2023 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This introduces a breaking change enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants