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.

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

Release 1.2.1 #55

Merged
merged 3 commits into from
Nov 1, 2024
Merged

Release 1.2.1 #55

merged 3 commits into from
Nov 1, 2024

Conversation

Marius9595
Copy link
Collaborator

This pull request includes several changes to the @leanmind/monads package, focusing on improving error handling in the Try monad and updating the build configuration. The most important changes include updating the package.json to support different module formats, modifying the Failure class to provide more detailed error information, and updating the tests accordingly.

Build Configuration Updates:

  • package.json: Changed the main and module fields to point to CommonJS and ES module outputs respectively, and updated the build scripts to compile both module formats.

Error Handling Improvements:

  • src/try/try.ts: Modified the Try.from method to always create a new Failure instance with a detailed error message when no error is provided.
  • src/try/try.ts: Updated the Failure class to use a default type of Error and removed the static NO_ERROR_PROVIDED instance. [1] [2]

Test Updates:

  • src/try/try.test.ts: Updated the test cases to reflect the changes in the Failure class, ensuring that a new Failure instance is created with a detailed error message.

@Marius9595 Marius9595 requested a review from myugen November 1, 2024 17:47
@Marius9595 Marius9595 merged commit e2af08e into main Nov 1, 2024
7 checks passed
Copy link

github-actions bot commented Nov 1, 2024

🎉 This PR is included in version 1.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 1.3.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant