Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Cloud: change variable name #8590

Merged
merged 2 commits into from
Jan 29, 2021
Merged

Cloud: change variable name #8590

merged 2 commits into from
Jan 29, 2021

Conversation

hguthrie
Copy link
Contributor

Purpose of this pull request

This pull request (PR) corrects a variable name used in an example. I changed the variable name from branch to environment, which is a combination of branch name and environment ID...but very generic. When you log in to the remote server, you are greeted with This is environment <branchname-environmentID>. Together, they represent a kind of machine name that identifies the environment uniquely.

I looked for other instances, but didn't see any that use a variable in place of actual.

Affects the following topic:
https://devdocs.magento.com/cloud/project/manage-disk-space.html

@hguthrie hguthrie added Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies 2.x Internal Dev Differentiates work between community and Magento staff Small Changes labels Jan 29, 2021
@hguthrie hguthrie requested a review from bdenham January 29, 2021 17:52
@hguthrie hguthrie self-assigned this Jan 29, 2021
@dobooth
Copy link
Contributor

dobooth commented Jan 29, 2021

running tests

@dobooth dobooth enabled auto-merge January 29, 2021 20:31
@dobooth dobooth merged commit a927042 into master Jan 29, 2021
@dobooth dobooth deleted the hg-cloud-fixid branch January 29, 2021 20:45
@ghost
Copy link

ghost commented Jan 29, 2021

Hi @hguthrie, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies Internal Dev Differentiates work between community and Magento staff Progress: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants