Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 2, 2025

Summary

This PR updates the README with instructions for configuring NuGet credentials on Windows, addressing the issue raised in #17540 (comment).

Problem

Users attempting to build Cyberduck on Windows encounter 401/403 authentication errors when NuGet tries to access GitHub Package Registry sources (gh-ikvmnet and gh-iterate-ch). Without proper credentials configured, the build fails with cryptic error messages, blocking contributors from building the project locally.

Solution

Added a new "NuGet Configuration" section to the Windows prerequisites that provides clear, step-by-step instructions:

  1. How to create a GitHub personal access token (classic) with read:packages permissions
  2. Commands to configure NuGet sources with the credentials:
    dotnet nuget update source gh-ikvmnet -u "YourUsername" -p "YourPAT"
    dotnet nuget update source gh-iterate-ch -u "YourUsername" -p "YourPAT"
  3. What values need to be replaced (GitHub username and PAT)

The section is placed logically in the Windows prerequisites, after "System Configuration" and before the "Building" section, ensuring users complete this setup before attempting to build.

Related

Original prompt

This section details on the original issue you should resolve

<issue_title>Update build instructions in README with configuration for NuGet</issue_title>
<issue_description>As discussed in this comment 1, the README for the project should be amended with build instructions for Windows that include properly setting credentials for nuget with a personal access token.

Comments on the Issue (you are @copilot in this section)

Fixes #17541

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Footnotes

  1. https://github.com/iterate-ch/cyberduck/pull/17540#issuecomment-3361537789</issue_description>

Copy link

cla-assistant bot commented Oct 2, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
Copy link

cla-assistant bot commented Oct 2, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Copilot Copilot AI changed the title [WIP] Update build instructions in README with configuration for NuGet Update README with NuGet configuration instructions for Windows builds Oct 2, 2025
@Copilot Copilot AI requested a review from dkocher October 2, 2025 16:31
Copilot finished work on behalf of dkocher October 2, 2025 16:31
@dkocher dkocher requested a review from AliveDevil October 2, 2025 16:39
@dkocher dkocher marked this pull request as ready for review October 2, 2025 16:39
@dkocher dkocher requested a review from a team as a code owner October 2, 2025 16:39
@AliveDevil
Copy link
Contributor

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card!

This desperation.

@dkocher dkocher merged commit d16449e into master Oct 6, 2025
7 of 8 checks passed
@dkocher dkocher deleted the copilot/fix-59bb7920-3720-4cbe-bad6-c27c6d71f946 branch October 6, 2025 07:34
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.

Update build instructions in README with configuration for NuGet
3 participants