Skip to content

Conversation

@dvdplm
Copy link
Contributor

@dvdplm dvdplm commented Dec 8, 2020

…and some random nitpicks.

Diff is pretty noisy. Mergin master into aj-bounds likely helps with that.

@ascjones ascjones merged commit 1183560 into aj-bounds Dec 8, 2020
@ascjones ascjones deleted the dp-aj-bounds branch December 8, 2020 16:20
dvdplm added a commit that referenced this pull request Dec 14, 2020
* Add bounds for generic type param

* Fmt

* Remove redundant clone

* Make clippy happy

* Fmt

* Remove readding of type to bounds

* Unused imports

* Doc tweaks (#37)

* Docs and nits

* Update derive/src/trait_bounds.rs

Co-authored-by: Andrew Jones <[email protected]>

* Adapt and simplify code to scale-info's needs
Add a few trybuild tests (WIP)

* Resolve todo

* Fmt

* Add ui test for Unions

* Only run trybuild-tests on nightly

* Unify and simply collect_types_to_bind()

* Move a few trivial tests to trybuild tests instead

* Add more trybuild tests

* remove trivial test
add trybuild test files

* Make type_contains_idents more self-contained

* Obey the fmt

Co-authored-by: David <[email protected]>
dvdplm added a commit that referenced this pull request Jan 18, 2021
* Add bounds for generic type param

* Fmt

* Remove redundant clone

* Make clippy happy

* Fmt

* Remove readding of type to bounds

* Unused imports

* Doc tweaks (#37)

* Docs and nits

* Update derive/src/trait_bounds.rs

Co-authored-by: Andrew Jones <[email protected]>

* Adapt and simplify code to scale-info's needs
Add a few trybuild tests (WIP)

* Resolve todo

* Fmt

* Add ui test for Unions

* Only run trybuild-tests on nightly

* Unify and simply collect_types_to_bind()

* Move a few trivial tests to trybuild tests instead

* Add more trybuild tests

* remove trivial test
add trybuild test files

* Make type_contains_idents more self-contained

* Obey the fmt

* WIP

* Cleanup

* cleanup

* cleanup

* Sort out funny whitespace in type names with lifetimes, `& \'static` –> `&'static`

* Impl review feedback

* Avoid clones

Co-authored-by: Andrew Jones <[email protected]>
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