search

dapr / dashboard

General purpose dashboard for Dapr
Apache License 2.0
181 stars 47 forks source link

feat: upgrade to go 1.21 and bump deps #273

Closed toneill818 closed 7 months ago

toneill818 commented 10 months ago

Upgrade to go 1.21 and bump dependencies

yaron2 commented 8 months ago

@toneill818 can you please check why the build is failing?

yaron2 commented 7 months ago

Ping @toneill818

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 27.56%. Comparing base (298a16a) to head (16f305d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #273 +/- ## ======================================= Coverage 27.56% 27.56% ======================================= Files 7 7 Lines 798 798 ======================================= Hits 220 220 Misses 554 554 Partials 24 24 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

toneill818 commented 7 months ago

@yaron2 Builds seem to be failing due to an old node version. Is it okay if I modify the node version for the runners in this PR?

The Angular CLI requires a minimum Node.js version of v18.13.

yaron2 commented 7 months ago

@yaron2 Builds seem to be failing due to an old node version. Is it okay if I modify the node version for the runners in this PR?

The Angular CLI requires a minimum Node.js version of v18.13.

Sure

toneill818 commented 7 months ago

@yaron2 Builds seem to be failing due to an old node version. Is it okay if I modify the node version for the runners in this PR?

The Angular CLI requires a minimum Node.js version of v18.13.

Sure

Should be fixed

yaron2 commented 7 months ago

@yaron2 Builds seem to be failing due to an old node version. Is it okay if I modify the node version for the runners in this PR?

The Angular CLI requires a minimum Node.js version of v18.13.

Sure

Should be fixed

Thanks!