Skip to content

chore: Update ROADMAP.md #7115

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Feb 16, 2022
Merged

chore: Update ROADMAP.md #7115

merged 4 commits into from
Feb 16, 2022

Conversation

tejal29
Copy link
Contributor

@tejal29 tejal29 commented Feb 15, 2022

Update Skaffold 2022 Roadmap.

@tejal29 tejal29 requested a review from a team as a code owner February 15, 2022 21:24
@tejal29 tejal29 requested a review from nkubala February 15, 2022 21:24
@tejal29 tejal29 marked this pull request as draft February 15, 2022 21:32
@codecov
Copy link

codecov bot commented Feb 15, 2022

Codecov Report

Merging #7115 (13b1336) into main (290280e) will decrease coverage by 1.83%.
The diff coverage is 56.96%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7115      +/-   ##
==========================================
- Coverage   70.48%   68.65%   -1.84%     
==========================================
  Files         515      558      +43     
  Lines       23150    26084    +2934     
==========================================
+ Hits        16317    17907    +1590     
- Misses       5776     6944    +1168     
- Partials     1057     1233     +176     
Impacted Files Coverage Δ
cmd/skaffold/app/cmd/deploy.go 52.00% <ø> (-1.85%) ⬇️
cmd/skaffold/app/cmd/dev.go 84.61% <0.00%> (ø)
cmd/skaffold/app/cmd/render.go 36.66% <0.00%> (-4.72%) ⬇️
cmd/skaffold/skaffold.go 0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/inspect_tests.go 62.50% <14.28%> (-1.14%) ⬇️
cmd/skaffold/app/cmd/lsp.go 28.12% <28.12%> (ø)
cmd/skaffold/app/cmd/fix.go 68.85% <40.00%> (-7.62%) ⬇️
cmd/skaffold/app/cmd/lint.go 42.85% <42.85%> (ø)
cmd/skaffold/app/cmd/find_configs.go 48.88% <50.00%> (+0.24%) ⬆️
cmd/skaffold/app/skaffold.go 76.19% <70.00%> (-8.43%) ⬇️
... and 208 more

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 35ec779...13b1336. Read the comment docs.

Copy link
Contributor

@MarlonGamez MarlonGamez left a comment

Choose a reason for hiding this comment

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

These look good to me, I just noticed one typo

Co-authored-by: Marlon Gamez <[email protected]>
@MarlonGamez
Copy link
Contributor

Not sure why the CLA check is failing. Any clue @tejal29 ?

@tejal29
Copy link
Contributor Author

tejal29 commented Feb 16, 2022

Not sure why the CLA check is failing. Any clue @tejal29 ?

yeah. Since I accepted your suggestions, the bot fails to check when two or more authors have commits.
We can manually resolve it.

@tejal29 tejal29 marked this pull request as ready for review February 16, 2022 19:32
@tejal29 tejal29 merged commit 9b9b7f2 into main Feb 16, 2022
@tejal29 tejal29 deleted the tejal29-patch-1 branch February 16, 2022 19:33
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.

2 participants