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

feat: Include aliases toBeSameAs and toBe for toBeSame #109

Merged
merged 3 commits into from
Oct 31, 2023

Conversation

CristhianMotoche
Copy link
Contributor

@CristhianMotoche CristhianMotoche commented Oct 24, 2023

Changes

Include toBeSameAs and toBe alias for toBeSame.

NOTE: My code formatter modified some files. Let me know if it's okay to keep it that way, otherwise I can revert that to keep only the necessary changes.

Solves #108


As part of Stack Builders Inc. Hacktoberfest 2023 event.

@@ -472,6 +471,28 @@ describe("[Unit] Assertion.test.ts", () => {
});
});

describe(".toBeSameAs", () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are the changes that matter in this file for this PR.

@@ -440,6 +444,48 @@ export class Assertion<T> {
});
}

/**
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are the changes that matter in this file for this PR.

Copy link
Member

@JoseLion JoseLion left a comment

Choose a reason for hiding this comment

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

The relevant changes look great! Do you think we can revert the formatter changes, though? 🙂

@CristhianMotoche
Copy link
Contributor Author

Hi @JoseLion I reverted the code formatter changes. Let me know if looks better now.

Copy link
Member

@JoseLion JoseLion left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for your contribution @CristhianMotoche 🎉

@JoseLion JoseLion merged commit c7a061a into main Oct 31, 2023
5 checks passed
@JoseLion JoseLion deleted the fix/solve-issue-108 branch October 31, 2023 21:14
@JoseLion JoseLion added the hacktoberfest-accepted PRs accepted during Hacktoberfest label Oct 31, 2023
Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 1.0.0 🎉

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
Labels
hacktoberfest-accepted PRs accepted during Hacktoberfest released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants