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

[TASK] Separate CGL tools from root composer.json #222

Merged
merged 1 commit into from
May 27, 2024

Conversation

eliashaeussler
Copy link
Owner

This PR separates all CGL tools from root composer.json. This makes dependency updates better maintainable. For contributors, two new DDEV commands ddev cgl and ddev test are added. They serve as bridge between the appropriate composer.json files and the defined Composer scripts.

@eliashaeussler eliashaeussler added the maintenance Code base is being maintained label May 27, 2024
@eliashaeussler eliashaeussler self-assigned this May 27, 2024
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 13295 lines exceeds the maximum allowed for the inline comments feature.

Copy link

codeclimate bot commented May 27, 2024

Code Climate has analyzed commit ea5e7b5 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 95.4% (0.0% change).

View more on Code Climate.

@coveralls
Copy link
Collaborator

coveralls commented May 27, 2024

Pull Request Test Coverage Report for Build 9253617899

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.438%

Totals Coverage Status
Change from base Build 9252399353: 0.0%
Covered Lines: 774
Relevant Lines: 811

💛 - Coveralls

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 13240 lines exceeds the maximum allowed for the inline comments feature.

@eliashaeussler eliashaeussler merged commit 36537cb into main May 27, 2024
19 checks passed
@eliashaeussler eliashaeussler deleted the task/separate-cgl branch May 27, 2024 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Code base is being maintained
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants