Skip to content

chore: separate pub use from regular use with #[rustfmt::skip]#1638

Merged
Benoît Cortier (CBenoit) merged 3 commits into
masterfrom
copilot/separate-pub-use-statements
Dec 19, 2025
Merged

chore: separate pub use from regular use with #[rustfmt::skip]#1638
Benoît Cortier (CBenoit) merged 3 commits into
masterfrom
copilot/separate-pub-use-statements

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 18, 2025

  • Understand the issue and identify all files with pub use statements
  • Add #[rustfmt::skip] annotations to separate pub use from regular use statements in:
    • devolutions-gateway/src/ai/mod.rs
    • devolutions-gateway/src/plugin_manager/mod.rs
    • crates/devolutions-agent-shared/src/lib.rs
    • crates/devolutions-agent-shared/src/windows/mod.rs
    • crates/devolutions-pedm/src/lib.rs
    • crates/jmux-proto/src/lib.rs
    • crates/network-monitor/src/lib.rs
    • crates/jet-proto/src/lib.rs
    • crates/transport/src/lib.rs
    • crates/network-scanner/src/interfaces/mod.rs
    • crates/network-scanner/src/broadcast/mod.rs
    • crates/video-streamer/src/lib.rs
    • crates/proxy-socks/src/lib.rs
    • crates/jmux-proxy/src/lib.rs
    • crates/traffic-audit-libsql/src/lib.rs
    • crates/devolutions-pedm-shared/devolutions-pedm-client-http/src/models/mod.rs (reverted - no regular use statements)
    • crates/devolutions-pedm-shared/devolutions-pedm-client-http/src/apis/mod.rs
    • crates/devolutions-pedm-shared/src/client.rs
    • crates/win-api-wrappers/src/lib.rs
    • crates/win-api-wrappers/src/identity/sid.rs
    • crates/win-api-wrappers/src/str.rs
    • crates/job-queue-libsql/src/lib.rs
  • Run cargo +nightly fmt --all to verify the annotations work correctly
  • Run cargo +nightly fmt --all --check to ensure no formatting issues remain
  • Run clippy to ensure no issues introduced (passed without warnings)
  • Run code review (no issues found)
  • Security checks (CodeQL timed out, but no logic changes were made - only annotations added)
  • Address review feedback: removed unnecessary annotations from models/mod.rs
Original prompt

Build upon: #1637

Please, make sure that pub use are separated from regular use by using the annotation #[rustfmt::skip]. Ensure this is working by running cargo +nightly fmt.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

… statements

Co-authored-by: CBenoit <3809077+CBenoit@users.noreply.github.com>
Copilot AI changed the title [WIP] Ensure pub use is separated with rustfmt skip chore: separate pub use from regular use with #[rustfmt::skip] Dec 18, 2025
Co-authored-by: CBenoit <3809077+CBenoit@users.noreply.github.com>
auto-merge was automatically disabled December 18, 2025 12:46

Head branch was pushed to by a user without write access

@CBenoit Benoît Cortier (CBenoit) merged commit c3fd3f1 into master Dec 19, 2025
102 of 120 checks passed
@CBenoit Benoît Cortier (CBenoit) deleted the copilot/separate-pub-use-statements branch December 19, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants