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.

Sign up for GitHub

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

Jump to bottom

Formatted composer.json using JQ --indent 4 #109

Merged
merged 1 commit into from
Mar 18, 2024
Merged

Formatted composer.json using JQ --indent 4 #109

merged 1 commit into from
Mar 18, 2024

Conversation

ibexa-yuna
Copy link
Contributor

@ibexa-yuna ibexa-yuna commented Mar 15, 2024 •
edited by alongosz
Loading

Question Answer
JIRA issue n/a
Type improvement
Target Ibexa version v4.5+
BC breaks no

Aligned formatting of composer.json, for consistency's sake with the pre-existing conventions, ease of merge-ups and task automation.

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly.
  • Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
  • Asked for a review.

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Mar 15, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@alongosz alongosz merged commit 259ed71 into 4.5 Mar 18, 2024
14 checks passed
@alongosz alongosz deleted the reformat-composer branch March 18, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomaszszopinski tomaszszopinski tomaszszopinski approved these changes

@micszo micszo micszo approved these changes

Assignees

@micszo micszo

@katarzynazawada katarzynazawada

@tomaszszopinski tomaszszopinski

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ibexa-yuna @micszo @tomaszszopinski @alongosz @katarzynazawada