Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

Renamed global preference file to preference.yaml from config.yaml#1784

Merged
openshift-merge-robot merged 1 commit intoredhat-developer:masterfrom
girishramnani:rename/global_config_to_preference
Jun 7, 2019
Merged

Renamed global preference file to preference.yaml from config.yaml#1784
openshift-merge-robot merged 1 commit intoredhat-developer:masterfrom
girishramnani:rename/global_config_to_preference

Conversation

@girishramnani
Copy link
Copy Markdown
Contributor

What is the purpose of this change? What does it change?

The global preference's default name changes to preference.yaml instead of config.yaml. As this file is present in the ~/.odo folder it will not colide with the config.yaml if someone chooses to create a component in home dir.

Was the change discussed in an issue?

fixes #1766

How to test changes?

odo preference set Timeout 5 # this should create a ~/.odo/preference.yaml instead of ~/.odo/config.yaml

NOTE - Add this to the ChangeLog as this is a breaking change

@cdrage
Copy link
Copy Markdown
Member

cdrage commented Jun 4, 2019

/retest

Copy link
Copy Markdown
Member

@cdrage cdrage left a comment

Choose a reason for hiding this comment

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

This LGTM.

Are there any tests you need to fix?

@girishramnani
Copy link
Copy Markdown
Contributor Author

/retest

@girishramnani
Copy link
Copy Markdown
Contributor Author

No all the tests should work @cdrage

@cdrage
Copy link
Copy Markdown
Member

cdrage commented Jun 4, 2019

/retest

@kadel
Copy link
Copy Markdown
Member

kadel commented Jun 5, 2019

@girishramnani We are not doing ensuring the backward compatibility after all?

@girishramnani
Copy link
Copy Markdown
Contributor Author

@kadel according to the discussion in the issue, @jorgemoralespou said that having a changelog entry would be enough

After this commit, the global preference's default name will be preference.yaml instead of config.yaml. As this file is present in the ~/.odo folder it will not colide with the config.yaml if someone chooses to create a component in home dir.
@kadel
Copy link
Copy Markdown
Member

kadel commented Jun 7, 2019

@girishramnani can you please add how will the Release change entry look like?

@kadel according to the discussion in the issue, @jorgemoralespou said that having a changelog entry would be enough

I know, I was saying originally the same :-)
I'm ok with this.
/approve

I'm just making sure that this is what you want to do as you had good arguments why we should try to do it as nonbreaking change.

@kadel
Copy link
Copy Markdown
Member

kadel commented Jun 7, 2019

/approve

@openshift-ci-robot
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kadel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. label Jun 7, 2019
@cdrage
Copy link
Copy Markdown
Member

cdrage commented Jun 7, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Jun 7, 2019
@openshift-merge-robot openshift-merge-robot merged commit 68c2928 into redhat-developer:master Jun 7, 2019
@rm3l rm3l added the estimated-size/XS (1-5) Rough sizing for Epics. Less than one sprint of work for one person, but smaller than S. label Jun 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. estimated-size/XS (1-5) Rough sizing for Epics. Less than one sprint of work for one person, but smaller than S. lgtm Indicates that a PR is ready to be merged. Required by Prow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cannot create component in home directory

6 participants