Skip to content
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

🌱 Upgrade to go 1.18 #2143

Merged
merged 9 commits into from
Aug 17, 2022
Merged

🌱 Upgrade to go 1.18 #2143

merged 9 commits into from
Aug 17, 2022

Conversation

naveensrinivasan
Copy link
Member

  • Upgrade to go 1.18
  • Updated the deps to avoid critical CVE's

Signed-off-by: naveensrinivasan 172697+naveensrinivasan@users.noreply.github.com

@codecov
Copy link

codecov bot commented Aug 12, 2022

Codecov Report

Merging #2143 (7e2411f) into main (887facf) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@           Coverage Diff           @@
##             main    #2143   +/-   ##
=======================================
  Coverage   42.38%   42.39%           
=======================================
  Files          94       94           
  Lines        7825     7824    -1     
=======================================
  Hits         3317     3317           
+ Misses       4250     4249    -1     
  Partials      258      258           

Copy link
Contributor

@azeemshaikh38 azeemshaikh38 left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@azeemshaikh38
Copy link
Contributor

Let's use the GO_VERSION env for all GitHub workflows?

@naveensrinivasan naveensrinivasan force-pushed the naveen/feat/1.18 branch 2 times, most recently from 55fa94f to a28d710 Compare August 12, 2022 20:43
@naveensrinivasan naveensrinivasan enabled auto-merge (squash) August 13, 2022 05:17
@naveensrinivasan naveensrinivasan force-pushed the naveen/feat/1.18 branch 2 times, most recently from 918ae5c to 20d83e5 Compare August 13, 2022 05:23
- Upgrade to go 1.18
- Updated the deps to avoid critical CVE's

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
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.

2 participants