Skip to content
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

[BUGS-7678] add PANTHEON_HOSTNAME to application.pantheon.php (#119) #120

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

jazzsequence
Copy link
Contributor

  • break up the PANTHEON_ENVIRONMENT conditional

so we can use the same one later for the new stuff we're adding

  • define PANTHEON_HOSTNAME if it's not already defined

  • add phpcbf command

  • lint

  • add our filter to update the config instructions use Config::define instead of define

  • return the contents

  • apply changes from wp upstream

* break up the PANTHEON_ENVIRONMENT conditional

so we can use the same one later for the new stuff we're adding

* define PANTHEON_HOSTNAME if it's not already defined

* add phpcbf command

* lint

* add our filter to update the config instructions
use Config::define instead of define

* return the contents

* apply changes from wp upstream
@jazzsequence jazzsequence requested review from a team as code owners April 30, 2024 17:51
@jazzsequence jazzsequence merged commit 5178b15 into default Apr 30, 2024
4 checks passed
@jazzsequence jazzsequence deleted the main branch April 30, 2024 17:52
jazzsequence pushed a commit that referenced this pull request Jul 31, 2024
jazzsequence added a commit that referenced this pull request Jul 31, 2024
…120)

* break up the PANTHEON_ENVIRONMENT conditional

so we can use the same one later for the new stuff we're adding

* define PANTHEON_HOSTNAME if it's not already defined

* add phpcbf command

* lint

* add our filter to update the config instructions
use Config::define instead of define

* return the contents

* apply changes from wp upstream
jazzsequence pushed a commit that referenced this pull request Aug 1, 2024
jazzsequence added a commit that referenced this pull request Aug 1, 2024
…120)

* break up the PANTHEON_ENVIRONMENT conditional

so we can use the same one later for the new stuff we're adding

* define PANTHEON_HOSTNAME if it's not already defined

* add phpcbf command

* lint

* add our filter to update the config instructions
use Config::define instead of define

* return the contents

* apply changes from wp upstream
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant