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.

Sign up for GitHub

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

Jump to bottom

fix: change hash algorithm #12109

Merged
merged 1 commit into from
Jul 30, 2024
Merged

fix: change hash algorithm #12109

merged 1 commit into from
Jul 30, 2024

Conversation

jayzhang
Copy link
Member

@jayzhang jayzhang commented Jul 30, 2024 •
edited
Loading

@jayzhang jayzhang changed the title fix: hash algorithm fix: change hash algorithm Jul 30, 2024
@codecov Codecov
Copy link

codecov bot commented Jul 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.45%. Comparing base (f173633) to head (b36e7f7).
Report is 10 commits behind head on dev.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev   #12109      +/-   ##
==========================================
+ Coverage   89.42%   89.45%   +0.02%     
==========================================
  Files         570      570              
  Lines       33583    33555      -28     
  Branches     6573     6441     -132     
==========================================
- Hits        30033    30016      -17     
+ Misses       1586     1583       -3     
+ Partials     1964     1956       -8
Files Coverage Δ
packages/fx-core/src/common/globalState.ts 86.66% <ø> (ø)
...es/fx-core/src/core/middleware/concurrentLocker.ts 94.11% <ø> (ø)

... and 11 files with indirect coverage changes

@jayzhang jayzhang marked this pull request as ready for review July 30, 2024 07:11
@jayzhang jayzhang merged commit 2d21210 into dev Jul 30, 2024
28 checks passed
@jayzhang jayzhang deleted the huajie/fix-hash branch July 30, 2024 07:17
@github-actions github-actions bot mentioned this pull request Aug 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LongOddCode LongOddCode LongOddCode approved these changes

@nliu-ms nliu-ms Awaiting requested review from nliu-ms nliu-ms is a code owner

@xzf0587 xzf0587 Awaiting requested review from xzf0587 xzf0587 is a code owner

@hund030 hund030 Awaiting requested review from hund030 hund030 is a code owner

@tecton tecton Awaiting requested review from tecton tecton is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jayzhang @LongOddCode