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

Added small section about default values#11956

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:4.4fromj92:dotenv-defaults
Aug 6, 2019

Conversation

@j92
Copy link
Contributor

@j92j92 commentedJul 13, 2019

Added a short description and a code sample of how to use default values when referenced variables are not set. The feature PR issymfony/symfony#31546.

symfony-splitter pushed a commit to symfony/dotenv that referenced this pull requestAug 6, 2019
… not set (j92)This PR was squashed before being merged into the 4.4 branch (closes #31546).Discussion----------[Dotenv] Use default value when referenced variable is not set| Q             | A| ------------- | ---| Branch?       | 4.4| Bug fix?      | no| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->| BC breaks?    | no     <!-- seehttps://symfony.com/bc -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->| Fixed tickets | #...   <!-- #-prefixed issue number(s), if any -->| License       | MIT| Doc PR        |symfony/symfony-docs#11956 <!-- required for new features -->In bash you have the option to define a default variable like this:```bashFOO=${VARIABLE:-default}```When VARIABLE is not set```bashFOO=${VARIABLE:-default} #FOO=default```When VARIABLE is set:```bashVARIABLE=testFOO=${VARIABLE:-default} #FOO=test```If others find this also a good idea, I will write documentation and add the Doc PR. But first I would like some feedback to check if anyone agrees with this feature.Commits-------790dbad149 [Dotenv] Use default value when referenced variable is not set
xabbuh added a commit to symfony/symfony that referenced this pull requestAug 6, 2019
… not set (j92)This PR was squashed before being merged into the 4.4 branch (closes#31546).Discussion----------[Dotenv] Use default value when referenced variable is not set| Q             | A| ------------- | ---| Branch?       | 4.4| Bug fix?      | no| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->| BC breaks?    | no     <!-- seehttps://symfony.com/bc -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->| Fixed tickets | #...   <!-- #-prefixed issue number(s), if any -->| License       | MIT| Doc PR        |symfony/symfony-docs#11956 <!-- required for new features -->In bash you have the option to define a default variable like this:```bashFOO=${VARIABLE:-default}```When VARIABLE is not set```bashFOO=${VARIABLE:-default} #FOO=default```When VARIABLE is set:```bashVARIABLE=testFOO=${VARIABLE:-default} #FOO=test```If others find this also a good idea, I will write documentation and add the Doc PR. But first I would like some feedback to check if anyone agrees with this feature.Commits-------790dbad [Dotenv] Use default value when referenced variable is not set
@xabbuhxabbuh added Dotenv and removed Waiting Code MergeDocs for features pending to be merged labelsAug 6, 2019
@xabbuhxabbuh modified the milestones:next,4.4Aug 6, 2019
@xabbuh
Copy link
Member

@j92 Thank you for implementing this feature and writing the docs for it. Can you rebase here now that the code has been merged? :)

@j92
Copy link
ContributorAuthor

j92 commentedAug 6, 2019

@xabbuh Done and thanks!

@OskarStark
Copy link
Contributor

Thank you, I added a versionadded directive while merging.

@OskarStarkOskarStark merged commit1848e00 intosymfony:4.4Aug 6, 2019
OskarStark added a commit that referenced this pull requestAug 6, 2019
This PR was merged into the 4.4 branch.Discussion----------Added small section about default values<!--If your pull request fixes a BUG, use the oldest maintained branch that containsthe bug (seehttps://symfony.com/roadmap for the list of maintained branches).If your pull request documents a NEW FEATURE, use the same Symfony branch wherethe feature was introduced (and `master` for features of unreleased versions).-->Added a short description and a code sample of how to use default values when referenced variables are not set. The feature PR issymfony/symfony#31546.Commits-------1848e00 Added small section about default values
OskarStark added a commit that referenced this pull requestAug 6, 2019
* 4.4:  Add missing versionadded directive. refs#11956  Added small section about default values
OskarStark added a commit that referenced this pull requestAug 6, 2019
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@wouterjwouterjwouterj approved these changes

@OskarStarkOskarStarkOskarStark approved these changes

+1 more reviewer

@3imed-jaberi3imed-jaberi3imed-jaberi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

4.4

Development

Successfully merging this pull request may close these issues.

6 participants

@j92@xabbuh@OskarStark@wouterj@3imed-jaberi@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp