Skip to content

Conversation

jmcph4
Copy link
Contributor

@jmcph4 jmcph4 commented Jul 12, 2023

Issue Addressed

#4494

Proposed Changes

  • Remove explicit re-exports of various types to appease the new compiler lint

Additional Info

It seems warn(hidden_glob_reexports) is the main culprit.

@jmcph4 jmcph4 changed the title 4494 rust 1 73 break Remove hidden re-exports to appease Rust 1.73 Jul 12, 2023
@jmcph4 jmcph4 self-assigned this Jul 12, 2023
@jmcph4 jmcph4 added low-hanging-fruit Easy to resolve, get it before someone else does! v4.4.1 ETA August 2023 labels Jul 12, 2023
@jmcph4 jmcph4 added the ready-for-review The code is ready for review label Jul 12, 2023
Copy link
Member

@michaelsproul michaelsproul 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 handling that!

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Jul 12, 2023
@michaelsproul
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Jul 12, 2023
## Issue Addressed

#4494 

## Proposed Changes

 - Remove explicit re-exports of various types to appease the new compiler lint

## Additional Info

It seems `warn(hidden_glob_reexports)` is the main culprit.
@bors
Copy link

bors bot commented Jul 12, 2023

This PR was included in a batch that timed out, it will be automatically retried

bors bot pushed a commit that referenced this pull request Jul 12, 2023
## Issue Addressed

#4494 

## Proposed Changes

 - Remove explicit re-exports of various types to appease the new compiler lint

## Additional Info

It seems `warn(hidden_glob_reexports)` is the main culprit.
@bors
Copy link

bors bot commented Jul 12, 2023

@bors bors bot changed the title Remove hidden re-exports to appease Rust 1.73 [Merged by Bors] - Remove hidden re-exports to appease Rust 1.73 Jul 12, 2023
@bors bors bot closed this Jul 12, 2023
@jmcph4 jmcph4 mentioned this pull request Jul 12, 2023
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
 - Remove explicit re-exports of various types to appease the new compiler lint

It seems `warn(hidden_glob_reexports)` is the main culprit.
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
 - Remove explicit re-exports of various types to appease the new compiler lint

It seems `warn(hidden_glob_reexports)` is the main culprit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge. v4.4.1 ETA August 2023
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants