Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[FrameworkBundle][HttpKernel] Introduce$buildDir argument toWarmableInterface::warmup to warm read-only artefacts inbuild_dir#19041

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
OskarStark merged 1 commit intosymfony:6.4fromalexandre-daubois:warmup-build-dir
Oct 18, 2023

Conversation

alexandre-daubois
Copy link
Member

If you don't need to preload anything, return an empty array.
If you don't need to preload anything, return an empty array. If read-only
artefacts need to be created, you can store them in a different directory
with the ``buildDir`` argument of the ``warmUp()`` method.
Copy link
Contributor

@OskarStarkOskarStarkOct 18, 2023
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Suggested change
with the ``buildDir`` argument of the ``warmUp()`` method.
with the ``$buildDir`` argument of the ``warmUp()`` method.

Not sure if we should use argument or parameter here

Copy link
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Let's use parameter everywhere 🙂


.. versionadded:: 6.4

The ``buildDir`` argument of the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Suggested change
The ``buildDir``argument of the
The ``$buildDir``parameter of the

Copy link
Contributor

@OskarStarkOskarStark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

A function signature contains parameters and is called with arguments

…ableInterface::warmup` to warm read-only artefacts in `build_dir`
@alexandre-daubois
Copy link
MemberAuthor

Good rule of thumb, I always get the two mixed up 😅

@OskarStark
Copy link
Contributor

Thank you Alexandre.

@OskarStarkOskarStark merged commitb1b12fa intosymfony:6.4Oct 18, 2023
@alexandre-dauboisalexandre-daubois deleted the warmup-build-dir branchOctober 18, 2023 09:51
@Okhoshi
Copy link
Contributor

Thank you@alexandre-daubois ! It was on my todo-list for today, but you beat me to it 😅

@alexandre-daubois
Copy link
MemberAuthor

Oops, my bad! I'll ask you next time so you can take the time to do it 🙂

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

@OskarStarkOskarStarkOskarStark approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
6.4
Development

Successfully merging this pull request may close these issues.

[FrameworkBundle][HttpKernel] Introduce$buildDir argument toWarmableInterface::warmup to warm read-only artefacts inbuild_dir
4 participants
@alexandre-daubois@OskarStark@Okhoshi@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp