forked from laravel/laravel
-
Notifications
You must be signed in to change notification settings - Fork 0
同步主仓库 #2
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
Open
wangbaojin
wants to merge
1,958
commits into
wangbaojin:master
Choose a base branch
from
laravel:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
同步主仓库 #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # CHANGELOG.md
Reorder
* add replace_placeholders to log channels * Update logging.php --------- Co-authored-by: Taylor Otwell <[email protected]>
* add batching config options to queue config file This adds the batching configuration options to the queue configuration skeleton, so everyone has a faster way of knowing that it's possible to customize the database connection and table options. * formatting --------- Co-authored-by: Taylor Otwell <[email protected]>
* remove rate limiter from route provider by default * remove policy place holder * remove broadcast skeleton in favor of new provider in core * use default provider collection * Remove unnecessary properties from exception handler. * add back broadcast provider * update comment * add rate limiting * Apply fixes from StyleCI * fix formatting --------- Co-authored-by: StyleCI Bot <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
* Preserve X-Xsrf-Token header from .htaccess Preserve X-Xsrf-Token header for session based authentication when building API in Laravel * Update .htaccess * Update .htaccess --------- Co-authored-by: Taylor Otwell <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
* Upgrade to Tailwind CSS v4.0 * Upgrade to Tailwind CSS 4 * Upgrade to Tailwind CSS 4 * Upgrade to Tailwind CSS 4 * Upgrade to Tailwind CSS 4 * Upgrade to Tailwind CSS v4.0 * remove compatible styles Co-authored-by: Julius Kiekbusch <[email protected]> * wip --------- Co-authored-by: Julius Kiekbusch <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
…public/index.php (#6531) * feat: add type hints for $app in artisan and public/index.php Added PHPDoc type hints for the $app variable to improve static analysis support in PHPStan and PHPStorm. This prevents tools from incorrectly flagging the variable and enhances developer experience. No functional changes; only improves code readability and IDE support. * Update artisan * Update index.php --------- Co-authored-by: Taylor Otwell <[email protected]>
force the timezone to be defined in the config file, rather than deferring to an environment variable. IMO having the timezone dependent on an environment variable adds an unnecessary amount of risk for your average user who may be unaware of the effects from changing this value. some scenarios where this could cause issues: - devs set this value to their local timezone on their local dev machines, and they are writing `Carbon:create()` commands or similar that now are working only under the assumption of that timezone value - you have multiple production servers setup across the country and each has their own timezone value according to their location, but they all talk to a central database. now the database is loaded with mixed timezone variables having an explicit value defined once for the application removes these risks. reverts part of #6188
* Re-add nunomaduro/collision * Update composer.json --------- Co-authored-by: Taylor Otwell <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Setting this to an empty string by default overrides the default behaviour of using APP_NAME in the cache config file, this leads to unintended consequences if this has been blindly copied into the app's main .env file without setting a value
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.