Skip to content

Display last output when missing the base command (7.4.x branch)#1490

Merged
Quetzacoalt91 merged 1 commit intoPrestaShop:7.4.xfrom
Quetzacoalt91:output-base-command-7.4.x
Oct 9, 2025
Merged

Display last output when missing the base command (7.4.x branch)#1490
Quetzacoalt91 merged 1 commit intoPrestaShop:7.4.xfrom
Quetzacoalt91:output-base-command-7.4.x

Conversation

@Quetzacoalt91
Copy link
Member

Questions Answers
Description? When encountering an issue with the installation of assets, I though this was caused by a bin/console that can't be called, but this actually caused by a module that was breaking the core.
Type? improvement
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? Delete bin/console during the update, we should expect the installation of assets to fail with some details.
Capture d’écran du 2025-10-08 17-09-48

@Quetzacoalt91 Quetzacoalt91 self-assigned this Oct 8, 2025
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Oct 8, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 8, 2025

@Quetzacoalt91 Quetzacoalt91 changed the title Display last output when missing the base command Display last output when missing the base command (7.4.x branch) Oct 9, 2025
@Quetzacoalt91
Copy link
Member Author

To be tested by the QA team with 7.4.1-beta.2

@Quetzacoalt91 Quetzacoalt91 merged commit 15eb0a1 into PrestaShop:7.4.x Oct 9, 2025
57 of 60 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Oct 9, 2025
@Quetzacoalt91 Quetzacoalt91 deleted the output-base-command-7.4.x branch October 9, 2025 12:56
@ps-jarvis
Copy link
Collaborator

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants