Skip to content

Conversation

@shivaahir158
Copy link
Collaborator

@shivaahir158 shivaahir158 commented Nov 22, 2024

Motivate of the pull request

  • To address an existing issue. If so, please provide a link to the issue:
  • Breaking new feature. If so, please describe details in the description part.

Describe the technical details -

Draft PR for Power and Thermal Specification block, the front end is ready, need to sort the API calling to FE code.

Note: this is an outdated PR, new one #277 has been up for this feature, I shall discard this when the newer one is merged into main branch.

image

Which part of the code base require a change

  • Frontend: powersummarytable.css, powersummarytable.js
  • Backend:
  • Library: <Specify the library name, e.g. npm packages>
  • Plug-in: <Specify the plugin name, e.g. Webpack, jtest>
  • Documentation
  • Regression tests
  • Continuous Integration (CI) scripts

Impact of the pull request

  • Require a change on Quality of Results (QoR)
  • Break backward-compatibility. If so, please list who may be influenced.
@codecov
Copy link

codecov bot commented Nov 22, 2024

Codecov Report

Attention: Patch coverage is 75.64103% with 19 lines in your changes missing coverage. Please review.

Project coverage is 68.83%. Comparing base (0c0cab8) to head (ad677ca).

Files with missing lines Patch % Lines
src/components/Tables/PowerSummaryTable.js 75.64% 19 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@ ## main #275 +/- ## ========================================== + Coverage 68.78% 68.83% +0.04%  ========================================== Files 110 110 Lines 9035 9086 +51 Branches 402 418 +16 ========================================== + Hits 6215 6254 +39  - Misses 2820 2832 +12 

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

@shivaahir158 shivaahir158 marked this pull request as ready for review January 28, 2025 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant