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

implement Eq for Package #172

Merged
merged 1 commit into from
Feb 16, 2022
Merged

implement Eq for Package #172

merged 1 commit into from
Feb 16, 2022

Conversation

MarcoIeni
Copy link
Contributor

No description provided.

@jyn514
Copy link
Collaborator

jyn514 commented Feb 13, 2022

When would this be useful?

@MarcoIeni
Copy link
Contributor Author

In https://github.com/MarcoIeni/release-plz I wanted to compare a package that I have edited locally with the same downloaded from crates.io.

@oli-obk
Copy link
Owner

oli-obk commented Feb 14, 2022

Are you interested in plain equality or will you need a diff anyway? In the latter case I would recommend serializing to json and using a json diff crate or tool

@MarcoIeni
Copy link
Contributor Author

For my use case I was just interested in equality. Anyway, apart from my use case, I submitted the PR because it's also suggested in the API guidelines.
Do you see any cons in implementing equality?

@oli-obk
Copy link
Owner

oli-obk commented Feb 15, 2022

Well... it's a huge type that seemed not useful for such traits to me. There's always some cost to adding such impls, but it's not relevant if there's a use case.

So i'll merge this, just gotta figure out the CI failure first

@ehuss
Copy link
Contributor

ehuss commented Feb 15, 2022

Sorry about the tests, I have opened #173 with the fixes.

@oli-obk oli-obk merged commit c3e1aac into oli-obk:main Feb 16, 2022
@oli-obk
Copy link
Owner

oli-obk commented Feb 16, 2022

published as 0.14.2

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.

5 participants