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 derive macro for num_traits::Signed #55

Merged
merged 3 commits into from
Oct 7, 2023

Conversation

tdelabro
Copy link
Contributor

@tdelabro tdelabro commented Jul 3, 2023

No description provided.

@tdelabro
Copy link
Contributor Author

tdelabro commented Jul 5, 2023

@cuviper Do you think this could be merged?

@cuviper
Copy link
Member

cuviper commented Jul 5, 2023

Would you be willing to write a corresponding newtype-derived Unsigned as well? That one is just a marker type, but it could still be useful to have #[derive(Unsigned)], just like you can with the Copy marker.

@tdelabro
Copy link
Contributor Author

tdelabro commented Jul 6, 2023

Sure. I will in the next days

@tdelabro
Copy link
Contributor Author

Hey, @cuviper. Sorry it took so long, I got really busy with my job.

Here it is tho :)

@tdelabro
Copy link
Contributor Author

tdelabro commented Oct 4, 2023

@cuviper ping

Copy link
Member

@cuviper cuviper left a comment

Choose a reason for hiding this comment

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

Just a couple of small things in the tests...

tests/newtype.rs Outdated Show resolved Hide resolved
tests/newtype.rs Outdated Show resolved Hide resolved
@cuviper
Copy link
Member

cuviper commented Oct 7, 2023

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 7, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 2c5013e into rust-num:master Oct 7, 2023
3 checks passed
@tdelabro tdelabro deleted the add-derive-Signed branch October 9, 2023 09:39
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.

2 participants