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

chore: Upgrade UI #100

Merged
merged 12 commits into from
Jun 19, 2024
Merged

chore: Upgrade UI #100

merged 12 commits into from
Jun 19, 2024

Conversation

ariefrahmansyah
Copy link
Contributor

@ariefrahmansyah ariefrahmansyah commented May 28, 2024

  • Update node js from 16 to 20
    • Add engines.node field in package.json across packages to pin this engines.node >=20.13.1
    • Update GitHub CI build-ui job to use node 20
  • Update React from 17 to 18
  • Update dependencies. Notable change: elastic ui
  • Update dev dependencies. Notable change: Craco and Rollup
  • Update GitHub CI jobs to use latest version of actions

@ariefrahmansyah ariefrahmansyah added the enhancement New feature or request label May 28, 2024
@ariefrahmansyah ariefrahmansyah self-assigned this May 28, 2024
Copy link

codecov bot commented May 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.39%. Comparing base (be7a48a) to head (232443f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   57.39%   57.39%           
=======================================
  Files          47       47           
  Lines        2354     2354           
=======================================
  Hits         1351     1351           
  Misses        827      827           
  Partials      176      176           
Flag Coverage Δ
api-test 57.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@leonlnj leonlnj left a comment

Choose a reason for hiding this comment

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

thanks! lgtm

@ariefrahmansyah ariefrahmansyah merged commit 688c2fc into main Jun 19, 2024
10 checks passed
@ariefrahmansyah ariefrahmansyah deleted the upgrade-ui branch June 19, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants