Skip to content

Conversation

@amisevsk
Copy link
Contributor

What does this PR do?

Renames .status.ideUrl to .status.mainUrl in DevWorkspaces.

What issues does this PR fix or reference?

Closes #370 (note we're focusing on the breaking change at the moment; adding additional info will still be possible down the line).

Is your PR tested? Consider putting some instruction how to test your changes

N/A

Docs PR

N/A

Since not all DevWorkspaces contain IDEs, it makes more sense to use a
generic name rather than the IDE-specific language.

Signed-off-by: Angel Misevski <[email protected]>
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: amisevsk, JPinkney, sleshchenko

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

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

@amisevsk amisevsk merged commit cd9c30e into devfile:main Apr 27, 2021
@amisevsk amisevsk deleted the rename-ideURL branch April 27, 2021 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extending DevWorkspace Status fields

5 participants