Skip to content

[WIP] Address feedback from review on pull request #279 - #282

Merged
ptr727 merged 2 commits into
developfrom
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged

[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom
copilot/sub-pr-279-another-one

Conversation

CopilotAI commented Jan 4, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor
  • Understand the specific feedback from review discussion #2659294065
  • Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  • Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  • Run tests to verify changes - all 10 tests pass
  • Request code review - passed with no issues

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

CopilotAI mentioned this pull request Jan 4, 2026
Merged
@ptr727
ptr727 marked this pull request as ready for review January 4, 2026 03:35
CopilotAI review requested due to automatic review settings January 4, 2026 03:35
CopilotAI reviewed Jan 4, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727 merged commit 2f0c71e into developJan 4, 2026
2 checks passed
@ptr727
ptr727 deleted the copilot/sub-pr-279-another-one branch January 4, 2026 03:42
CopilotAI requested a review from ptr727January 4, 2026 03:42
ptr727 added a commit that referenced this pull request Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727

Copilot code reviewCopilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
[WIP] Address feedback from review on pull request #279 by Copilot · Pull Request #282 · ptr727/Utilities · GitHub
Skip to content
[WIP] Address feedback from review on pull request #279 - #282
Merged
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Jan 4, 2026
Merged
[WIP] Address feedback from review on pull request #279#282
ptr727 merged 2 commits into
developfrom 
copilot/sub-pr-279-another-one
Conversation
CopilotAI
commented
Jan 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
  •  Understand the specific feedback from review discussion #2659294065
  •  Review FileExAsyncTests.cs line 162 for redundant ToString() calls
  •  Apply suggested changes - replaced Guid.NewGuid().ToString() with $"{Guid.NewGuid()}" throughout the file
  •  Run tests to verify changes - all 10 tests pass
  •  Request code review - passed with no issues
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
CopilotAI
mentioned this pull request
Jan 4, 2026
 Merged

@ptr727
ptr727
marked this pull request as ready for review
January 4, 2026 03:35
CopilotAI
review requested due to automatic review settings
January 4, 2026 03:35
CopilotAI
reviewed
Jan 4, 2026
View reviewed changes
CopilotAI
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@ptr727
Replace explicit ToString() calls with string interpolation
d7160ce
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
@ptr727
ptr727
merged commit 2f0c71e
into
developJan 4, 2026

2 checks passed

@ptr727
ptr727
deleted the

copilot/sub-pr-279-another-one

branch
January 4, 2026 03:42
CopilotAI
requested a review
from ptr727January 4, 2026 03:42
ptr727 added a commit
that referenced
this pull request
Jan 4, 2026
@ptr727@dependabot
@github-actions
Develop (#279)
df86e7c
* Bump the nuget-deps group with 3 updates
Bumps csharpier from 1.1.1 to 1.1.2
Bumps xunit.analyzers from 1.23.0 to 1.24.0
Bumps xunit.runner.visualstudio from 3.1.3 to 3.1.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.1.2
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
- dependency-name: xunit.analyzers
dependency-version: 1.24.0
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps xunit.runner.visualstudio from 3.1.4 to 3.1.5
---
updated-dependencies:
- dependency-name: xunit.runner.visualstudio
dependency-version: 3.1.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump csharpier from 1.1.2 to 1.2.1
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump Serilog.Sinks.Console from 6.0.0 to 6.1.1
---
updated-dependencies:
- dependency-name: Serilog.Sinks.Console
dependency-version: 6.1.1
dependency-type: direct:production
update-type: version-update:semver-minor
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Update dependencies
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.1 to 1.2.3
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.3
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the actions-deps group across 1 directory with 2 updates (#270)
Bumps the actions-deps group with 2 updates in the / directory: [actions/setup-dotnet](https://github.com/actions/setup-dotnet) and [actions/checkout](https://github.com/actions/checkout).
Updates `actions/setup-dotnet` from 4 to 5
- [Release notes](https://github.com/actions/setup-dotnet/releases)
- [Commits](actions/setup-dotnet@v4...v5)
Updates `actions/checkout` from 5 to 6
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-dotnet
dependency-version: '5'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
- dependency-name: actions/checkout
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
dependency-group: actions-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.3 to 1.2.4
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump the nuget-deps group with 1 update
Bumps csharpier from 1.2.4 to 1.2.5
---
updated-dependencies:
- dependency-name: csharpier
dependency-version: 1.2.5
dependency-type: direct:production
update-type: version-update:semver-patch
dependency-group: nuget-deps
...
Signed-off-by: dependabot[bot] <support@github.com>
* .NET 10 and AOT support
* Build pipeline also needs to be 10.0
* gitignore
* rider
* [WIP] Update changes based on feedback from pull request #279 (#280)
* Initial plan
* Add using statement and simplify CallerMemberName attributes
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Fix issues based on review feedback for pull request 279 (#281)
* Initial plan
* Make HttpClient timeout configurable via TimeoutSeconds property
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Update UtilitiesTests/DownloadAsyncTests.cs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] WIP address feedback from review on 'Develop' pull request (#283)
* Initial plan
* Replace redundant ToString() with string interpolation in FileExAsyncTests
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* [WIP] Address feedback from review on pull request #279 (#282)
* Initial plan
* Replace explicit ToString() calls with string interpolation
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
* Initial plan (#284)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Initial plan (#285)
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ptr727 <2061579+ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ptr727ptr727Awaiting requested review from ptr727

Assignees

@ptr727ptr727
Copilot code reviewCopilot

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@ptr727