Problem/Motivation

Normalize contents of composer.json for consistency.

Proposed resolution

composer normalize

Remaining tasks

- Review
- Merge

CommentFileSizeAuthor
#5 3157340-composer_normalize.patch4.85 KBxurizaemon
#2 sector-3180584-composer_normalize.patch5.17 KBxurizaemon

Comments

xurizaemon created an issue. See original summary.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
StatusFileSize
new sector-3180584-composer_normalize.patch5.17 KB

Normalized using ergebnis/composer-normalize, no functional changes.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented

This is going to conflict with #3157340: Remove committed WYSIWYG template library, happy to re-roll when the time is right. Or you can re-roll like this:

composer global require ergebnis/composer-normalize
cd /path/to/sector
composer normalize
danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Issue tags: +Needs reroll

This needs a reroll of the committed changes. Xuri would youj like to do that?

EDIT - Ah I just saw your comment about the conflict. OK I'll reroll when that is commited and pass on the credit to you. :)

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
StatusFileSize
new 3157340-composer_normalize.patch4.85 KB

Ah, here's a reroll but feel free to repeat or sing out if it needs doing again.

Glue work but nice to get things in order (and keep 'em that way with a CI step).

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Status: Active » Fixed

Thank you, I've decided to push this and then re-roll the other patch so it gives a cleaner diff.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.