Skip to content

provide context folder if no workspace present#950

Merged
mohitsuman merged 3 commits intoredhat-developer:masterfrom
mohitsuman:gh-923
Jul 25, 2019
Merged

provide context folder if no workspace present#950
mohitsuman merged 3 commits intoredhat-developer:masterfrom
mohitsuman:gh-923

Conversation

@mohitsuman
Copy link
Copy Markdown
Contributor

@mohitsuman mohitsuman commented Jul 22, 2019

fix #923 #951

@mohitsuman mohitsuman requested a review from dgolovin July 22, 2019 17:24
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 22, 2019

Codecov Report

Merging #950 into master will decrease coverage by 0.05%.
The diff coverage is 90.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #950      +/-   ##
==========================================
- Coverage   87.43%   87.37%   -0.06%     
==========================================
  Files          28       28              
  Lines        1846     1854       +8     
  Branches      313      317       +4     
==========================================
+ Hits         1614     1620       +6     
- Misses        232      234       +2
Impacted Files Coverage Δ
src/odo.ts 85.45% <100%> (+0.02%) ⬆️
src/openshift/component.ts 95.35% <88.88%> (-0.3%) ⬇️
src/util/watch.ts 96.55% <0%> (-3.45%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd1b56a...3d161b6. Read the comment docs.

Copy link
Copy Markdown
Collaborator

@dgolovin dgolovin left a comment

Choose a reason for hiding this comment

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

We should stick to the same strategy we have for cluster url, login and password. There should be + Select folder from local file system item in the list to allow add new folder to workspace.

@mohitsuman mohitsuman merged commit fd8adeb into redhat-developer:master Jul 25, 2019
@mohitsuman mohitsuman deleted the gh-923 branch June 16, 2022 19:24
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.

Cannot create component from workspace folder when workspace is empty

2 participants