Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix Dora score calculation and update DoraScore component #530

Merged
merged 10 commits into from
Sep 5, 2024

Conversation

e-for-eshaan
Copy link
Contributor

@e-for-eshaan e-for-eshaan commented Aug 30, 2024

This pull request includes several changes to fix the calculation of the Dora score and update the DoraScore component. The commits in this pull request address the following issues:

  • creates a new dora-score card: This commit creates a new DoraScoreV2 component that replaces the existing DoraScore component. The new component includes updated logic for calculating the Dora score and displays the score in a more visually appealing way.

  • fixes adaptations of deployment frequency metrics: This commit fixes the adaptations of the deployment frequency metrics in the DoraScore component. It ensures that the correct deployment frequency values are used in the score calculation.

  • removes redundant function: This commit removes a redundant function that was no longer needed in the DoraScore component.

  • fixes dora score: This commit fixes the calculation of the Dora score in the getDoraScore function. It addresses issues with the calculation w.r.t to deployment frequency

Screen.Recording.2024-09-02.at.6.01.35.PM.mov

These changes improve the accuracy of the Dora score calculation and update the DoraScore component to reflect the changes.

@e-for-eshaan e-for-eshaan marked this pull request as ready for review August 30, 2024 11:51
@e-for-eshaan e-for-eshaan merged commit 418662f into main Sep 5, 2024
1 check passed
@e-for-eshaan e-for-eshaan deleted the fix/dep-freq branch September 5, 2024 13:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants