Skip to content

chore(deps): remove hasha#7017

Merged
DanielSLew merged 1 commit intonetlify:mainfrom
v1rtl:remove-hasha
Jan 29, 2025
Merged

chore(deps): remove hasha#7017
DanielSLew merged 1 commit intonetlify:mainfrom
v1rtl:remove-hasha

Conversation

@v1rtl
Copy link
Contributor

@v1rtl v1rtl commented Jan 29, 2025

🎉 Thanks for submitting a pull request! 🎉

Summary

Fixes my mania for dependency minimalism

No need to use hasha, hashing strings and files is pretty straighftoward.


For us to review and ship your PR efficiently, please perform the following steps:

  • Open a bug/issue before writing your code 🧑‍💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
  • Read the contribution guidelines 📖. This ensures your code follows our style guide and
    passes our tests.
  • Update or add tests (if any source code was changed or added) 🧪
  • Update or add documentation (if features were changed or added) 📝
  • Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

N/A

Copy link
Contributor

@sarahetter sarahetter left a comment

Choose a reason for hiding this comment

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

Thank you!

@v1rtl
Copy link
Contributor Author

v1rtl commented Jan 29, 2025

a weird e2e error that is unrelated to my changes

https://github.com/netlify/cli/actions/runs/13022704468/job/36355017279?pr=7017#step:6:754

@DanielSLew DanielSLew merged commit 02f45d9 into netlify:main Jan 29, 2025
47 checks passed
@v1rtl v1rtl deleted the remove-hasha branch January 29, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants