Skip to content

Update to micro_kernel_trait.rst #10480

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

Merged
merged 3 commits into from
Nov 30, 2018
Merged

Update to micro_kernel_trait.rst #10480

merged 3 commits into from
Nov 30, 2018

Conversation

Alienpruts
Copy link
Contributor

Whenever you make a change to the "autoload' section of the composer.json file (amongst others), you need to regenerate the autoload files using composer dumpautoload, otherwise your changes will not be seen.

More info :
https://getcomposer.org/doc/03-cli.md#dump-autoload-dumpautoload-

Sorry if the proposed changes are not marked up properly, haven't gotten the hang of Markup language just yet :)

Whenever you make a change to the "autoload' section of the composer.json file (amongst others), you need to regenerate the autoload files using composer dumpautoload, otherwise your changes will not be seen.

More info :
https://getcomposer.org/doc/03-cli.md#dump-autoload-dumpautoload-

Sorry if the proposed changes are not marked up properly, haven't gotten the hang of Markup language just yet :)
@ndench
Copy link
Contributor

ndench commented Nov 14, 2018

Ping @Alienpruts could you apply the change suggested by @alexislefebvre?

Co-Authored-By: Alienpruts <vandecasteele@evilones.be>
@Alienpruts
Copy link
Contributor Author

Sorry, this seems to have slipped by my mailbox. Done

@xabbuh xabbuh added this to the 2.8 milestone Nov 15, 2018
@xabbuh xabbuh modified the milestones: 2.8, 3.4 Nov 28, 2018
@xabbuh xabbuh merged commit cca8833 into symfony:3.4 Nov 30, 2018
xabbuh added a commit that referenced this pull request Nov 30, 2018
…bvre)

This PR was merged into the 3.4 branch.

Discussion
----------

Update to micro_kernel_trait.rst

Whenever you make a change to the "autoload' section of the composer.json file (amongst others), you need to regenerate the autoload files using composer dumpautoload, otherwise your changes will not be seen.

More info :
https://getcomposer.org/doc/03-cli.md#dump-autoload-dumpautoload-

Sorry if the proposed changes are not marked up properly, haven't gotten the hang of Markup language just yet :)

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

cca8833 Update micro_kernel_trait.rst
fe2efd4 Update configuration/micro_kernel_trait.rst
aaa2b2b Update to micro_kernel_trait.rst
xabbuh added a commit that referenced this pull request Nov 30, 2018
@xabbuh
Copy link
Member

xabbuh commented Nov 30, 2018

Thank @Alienpruts and @alexislefebvre for these nice changes. FYI I have reused your wording from #10481 in 9827fea.

xabbuh added a commit that referenced this pull request Nov 30, 2018
* 3.4:
  Teach the user to write the command lazy loading
  [#10480] reuse wording from multi kernel article
  Update Multiple_kernels.rst
  Update usage.rst
  add annotation block to routing conditions
  Fix example for collection form type
  Update micro_kernel_trait.rst
  Update configuration/micro_kernel_trait.rst
  Update to micro_kernel_trait.rst
xabbuh added a commit that referenced this pull request Nov 30, 2018
* 4.1:
  Teach the user to write the command lazy loading
  [#10480] reuse wording from multi kernel article
  Update Multiple_kernels.rst
  Update usage.rst
  add annotation block to routing conditions
  fix the markup
  Clarifying what happens with --env=prod + APP_ENV=prod
  Fix example for collection form type
  Update micro_kernel_trait.rst
  Update configuration/micro_kernel_trait.rst
  Update to micro_kernel_trait.rst
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.

6 participants