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

Add convenience comparison operators for DeviceScreenType and RefinedSize #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tjarvstrand
Copy link
Contributor

No description provided.

@mcmah309
Copy link

@FilledStacks +1. It would also be nice if sometime soon we can get rid of the deprecated fields. It makes working with switch expressions annoying.

@tjarvstrand
Copy link
Contributor Author

@mcmah309 Sadly, this repo seems pretty dead.

@FilledStacks
Copy link
Owner

It's not 😅 we just don't have new features we need to add.

But we use it in all our projects.

I'll review this and merge in.

I maintain 11 open source packages so excuse my tardiness on some of them, I get a lot of requests, I do this alone, I fund it entirely myself from my business profits and I don't ask for any sponsorship or that anyone use it.

Copy link
Owner

@FilledStacks FilledStacks left a comment

Choose a reason for hiding this comment

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

I like this addition.

Can you please update the version to 0.8.0 and add an entry into the change log


final int _ordinal;

bool operator >(DeviceScreenType other) => _ordinal > other._ordinal;
Copy link
Owner

Choose a reason for hiding this comment

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

This is really cool

@tjarvstrand tjarvstrand force-pushed the add-convenience-comparison-operators branch from e9667c2 to f7a4745 Compare July 2, 2024 12:46
@tjarvstrand
Copy link
Contributor Author

Done!

@tjarvstrand tjarvstrand force-pushed the add-convenience-comparison-operators branch from f7a4745 to c9b2937 Compare October 20, 2024 12:53
@tjarvstrand
Copy link
Contributor Author

@FilledStacks Can this be merged now? :)

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