Skip to content

Conversation

ThomasLandauer
Copy link
Contributor

Page: https://symfony.com/doc/6.4/frontend/asset_mapper.html#importing-3rd-party-javascript-packages

Reason: The command was in the text and then again in the code block.

@carsonbot carsonbot added this to the 6.4 milestone Mar 4, 2024
@javiereguiluz javiereguiluz merged commit 28e6757 into symfony:6.4 Mar 4, 2024
@javiereguiluz
Copy link
Member

Thomas, I agree on removing the duplicated command in the text. That makes docs easier to maintain.

However, I don't fully agree with the proposed text and had to revert part of it while merging (See 4f253a5).

The problem is using the wording "To intialize your project..." because you need to run that command not only when initializing the project, but whenever someone adds new assets to the project and you fetch the changes from the repo.

@ThomasLandauer ThomasLandauer deleted the patch-25 branch March 4, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants