Skip to content

Clarify dotnet-quality behavior for multiple SDK versions - #780

Open
v-mahabaleshwars wants to merge 2 commits into
actions:mainfrom
v-mahabaleshwars:feature/778-dotnet-quality
Open

v-mahabaleshwars wants to merge 2 commits into
actions:mainfrom
v-mahabaleshwars:feature/778-dotnet-quality

Conversation

@v-mahabaleshwars

@v-mahabaleshwars v-mahabaleshwars commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Description:
Document that dotnet-quality applies to every dotnet-version in the same step. Add the recommended two-step workflow for installing a released SDK alongside a preview SDK.

Related issue:
778

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Copilot AI lite review requested due to automatic review settings September 1, 2026 16:56
@v-mahabaleshwars
v-mahabaleshwars requested a review from a team as a code owner September 1, 2026 16:56

Copilot AI 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.

🟡 Changes recommended

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the documentation for actions/setup-dotnet to clarify how the dotnet-quality input behaves when multiple SDK versions are installed in a single step, and provides a recommended workflow to install GA and preview SDKs together.

Changes:

  • Clarifies that dotnet-quality is shared across all dotnet-version entries within the same step (no per-version quality).
  • Adds a recommended two-step example for installing a released SDK alongside a preview SDK.
File summaries
File Description
README.md Documents dotnet-quality behavior across multiple versions and adds a two-step GA+preview installation example.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md Outdated
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.

3 participants