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

refactor(hash): Use 'Get-FileHash()' directly #5177

Merged
merged 2 commits into from
Oct 13, 2022
Merged

Conversation

niheaven
Copy link
Member

@niheaven niheaven commented Oct 10, 2022

Description

PowerShell 5 and later have Get-FileHash() and we could use it directly.

Motivation and Context

Simplify code base.

How Has This Been Tested?

scoop install and checkver

Checklist:

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.
  • I have added an entry in the CHANGELOG.

@niheaven niheaven merged commit 24301ac into develop Oct 13, 2022
@niheaven niheaven deleted the remove-compute-hash branch October 13, 2022 11:15
github-actions bot pushed a commit to pukkandan/Scoop that referenced this pull request Oct 20, 2022
# 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