Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

open func.yaml file when clicking on tree view #258

Merged
merged 1 commit into from
May 4, 2022

Conversation

sudhirverma
Copy link
Contributor

Fix: #191

@codecov-commenter
Copy link

codecov-commenter commented May 4, 2022

Codecov Report

Merging #258 (10cc69e) into main (a04c63d) will decrease coverage by 0.30%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
- Coverage   85.22%   84.91%   -0.31%     
==========================================
  Files          77       78       +1     
  Lines        3140     3156      +16     
  Branches      531      534       +3     
==========================================
+ Hits         2676     2680       +4     
- Misses        464      476      +12     
Impacted Files Coverage Δ
.../functions/function-tree-view/functionsTreeItem.ts 33.89% <20.00%> (-2.47%) ⬇️
...tions/function-command/open-yaml-file-in-editor.ts 30.00% <30.00%> (ø)
src/extension.ts 71.21% <50.00%> (-0.67%) ⬇️

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 a04c63d...10cc69e. Read the comment docs.

@sudhirverma sudhirverma requested a review from lstocchi May 4, 2022 09:31
Copy link
Contributor

@lstocchi lstocchi left a comment

Choose a reason for hiding this comment

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

LGTM

@sudhirverma sudhirverma merged commit 3ea7894 into redhat-developer:main May 4, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Functions: double-click on a Function in the list should open relevant func.yaml file
3 participants