Skip to content

fix(dvc-pipe-proxy): remove trailing punctuation from log messages#1380

Merged
Benoît Cortier (CBenoit) merged 2 commits into
masterfrom
copilot/fix-code-review-issues
Jun 23, 2026
Merged

fix(dvc-pipe-proxy): remove trailing punctuation from log messages#1380
Benoît Cortier (CBenoit) merged 2 commits into
masterfrom
copilot/fix-code-review-issues

Conversation

Copilot AI commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Log messages in ironrdp-dvc-pipe-proxy violated the STYLE.md rule against trailing periods/punctuation, flagged in the Copilot review on #1379.

  • worker.rs: stripped trailing . from 7 messages, trailing ... from "Reconnecting to DVC pipe...", and fixed a missing space after comma in a debug! macro call
  • platform/unix.rs: stripped trailing . from 2 messages

Copilot AI changed the title fix: remove trailing punctuation from log messages in dvc-pipe-proxy fix(dvc-pipe-proxy): remove trailing punctuation from log messages Jun 23, 2026
@CBenoit Benoît Cortier (CBenoit) marked this pull request as ready for review June 23, 2026 16:30
Comment thread crates/ironrdp-dvc-pipe-proxy/src/worker.rs Outdated
Comment thread crates/ironrdp-dvc-pipe-proxy/src/worker.rs Outdated
Co-authored-by: Benoît Cortier <3809077+CBenoit@users.noreply.github.com>
@CBenoit Benoît Cortier (CBenoit) enabled auto-merge (squash) June 23, 2026 16:32
@CBenoit Benoît Cortier (CBenoit) merged commit f385542 into master Jun 23, 2026
21 checks passed
@CBenoit Benoît Cortier (CBenoit) deleted the copilot/fix-code-review-issues branch June 23, 2026 16:49
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