Skip to content

Clarify the publishing articles by removing "deploy" where appropriate and merge similar info #47079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 31 commits into from
Aug 21, 2025
Merged

Clarify the publishing articles by removing "deploy" where appropriate and merge similar info #47079

merged 31 commits into from
Aug 21, 2025

Conversation

adegeo
Copy link
Contributor

@adegeo adegeo commented Jul 3, 2025 •
edited by github-actions bot
Loading

Summary

  • Tweaked the entire article to focus more on the environment where the app is run, versus a user running it on a computer.
  • Pros/Cons of each publish mode or option.
  • Changed a lot of the wording to remove "you" and "your" unless talking about examples the reader would do.
  • Merged the two different publishing articles into the overview article.
  • Talk about "Publish" as the action instead of "Deploy."
  • Add containers.
  • Linked to various related articles that talk about the modes.
  • When talking about running an app, used phrasing like "the host environment" to cover any kind of deployment after publishing (cloud or local).

Fixes #41489

Preview link: .NET application publishing overview

Important

docs/core/deploying/index.md is the file being reviewed. The other files are links and bookmarks updated to reflect merging the two other articles into this one.

Internal previews

Toggle expand/collapse
📄 File 🔗 Preview link
docs/architecture/microservices/multi-container-microservice-net-applications/multi-container-applications-docker-compose.md Defining your multi-container application with docker-compose.yml
docs/core/compatibility/sdk/6.0/apphost-generated-for-macos.md "Breaking change: Generate apphost for macOS"
docs/core/compatibility/sdk/8.0/arch-option.md --arch option doesn't imply self-contained
docs/core/compatibility/sdk/8.0/runtimespecific-app-default.md docs/core/compatibility/sdk/8.0/runtimespecific-app-default
docs/core/deploying/index.md .NET application publishing overview
docs/core/deploying/native-aot/index.md [Windows](https://review.learn.microsoft.com/en-us/dotnet/core/deploying/native-aot/index?branch=pr-en-us-47079)
docs/core/deploying/ready-to-run.md ReadyToRun deployment overview
docs/core/deploying/runtime-store.md Runtime package store
docs/core/deploying/single-file/overview.md Single-file deployment
docs/core/deploying/trimming/trim-self-contained.md Trim self-contained deployments and executables
docs/core/docker/build-container.md docs/core/docker/build-container
docs/core/install/macos-notarization-issues.md macOS Catalina Notarization and the impact on .NET downloads and projects
docs/core/native-interop/expose-components-to-com.md Expose .NET Core components to COM
docs/core/project-sdk/msbuild-props.md MSBuild properties for Microsoft.NET.Sdk
docs/core/rid-catalog.md .NET RID Catalog
docs/core/runtime-discovery/troubleshoot-app-launch.md Troubleshoot app launch failures
docs/core/tools/dotnet-clean.md docs/core/tools/dotnet-clean
docs/core/tools/dotnet-publish.md dotnet publish
docs/core/tools/sdk-errors/netsdk1071.md NETSDK1071: Explicitly versioned PackageReference to a metapackage that would be included with the framework.
docs/core/tools/troubleshoot-usage-issues.md Troubleshoot .NET tool usage issues
docs/core/tutorials/publishing-with-visual-studio-code.md Tutorial: Publish a .NET console application using Visual Studio Code
docs/core/tutorials/publishing-with-visual-studio.md docs/core/tutorials/publishing-with-visual-studio
docs/core/versions/selection.md Select the .NET version to use
docs/core/whats-new/dotnet-9/runtime.md What's new in .NET 9 runtime
docs/core/whats-new/dotnet-core-2-2.md What's new in .NET Core 2.2
docs/core/whats-new/dotnet-core-3-0.md What's new in .NET Core 3.0
docs/core/whats-new/dotnet-core-3-1.md docs/core/whats-new/dotnet-core-3-1
docs/navigate/devops-testing/toc.yml docs/navigate/devops-testing/toc
docs/navigate/tools-diagnostics/index.yml [linkListType: architecture
docs/standard/glossary.md .NET glossary

Note

This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.

@dotnetrepoman dotnetrepoman bot added this to the July 2025 milestone Jul 3, 2025
@BillWagner BillWagner modified the milestones: July 2025, August 2025 Aug 6, 2025
@japui-coder japui-coder mentioned this pull request Aug 11, 2025
Open
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

PoliCheck Scan Report

The following reports lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a comment that you post to the PR. The feedback helps reduce false positive instances in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

PoliCheck Scan Report

The following reports lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a comment that you post to the PR. The feedback helps reduce false positive instances in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

PoliCheck Scan Report

The following reports lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a comment that you post to the PR. The feedback helps reduce false positive instances in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@adegeo adegeo mentioned this pull request Aug 14, 2025
Closed
@adegeo adegeo marked this pull request as ready for review August 14, 2025 22:16
@adegeo adegeo changed the title Adegeo/41489/deploy Clarify the publishing articles by removing "deploy" where appropriate and merge similar info Aug 19, 2025
BillWagner
BillWagner approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM @adegeo

Let's :shipit:

gewarren
gewarren approved these changes Aug 20, 2025
View reviewed changes
@adegeo @gewarren
Apply suggestions from code review
6bd3f8b 
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@adegeo adegeo enabled auto-merge (squash) August 21, 2025 21:17
@adegeo adegeo merged commit 74a2221 into main Aug 21, 2025
10 checks passed
@adegeo adegeo deleted the adegeo/41489/deploy branch August 21, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BillWagner BillWagner BillWagner approved these changes

@gewarren gewarren gewarren approved these changes

@tdykstra tdykstra Awaiting requested review from tdykstra tdykstra is a code owner

@IEvangelist IEvangelist Awaiting requested review from IEvangelist IEvangelist is a code owner

@nishanil nishanil Awaiting requested review from nishanil nishanil is a code owner

@richlander richlander Awaiting requested review from richlander

Assignees
No one assigned
Labels
dotnet-architecture/svc dotnet-cli/subsvc dotnet-docker/subsvc dotnet-fundamentals/svc install/subsvc microservices/subsvc
Projects
None yet
Milestone
August 2025
Development

Successfully merging this pull request may close these issues.

(publish)confusing: Users of the application have to separately install the .NET runtime.
3 participants
@adegeo @BillWagner @gewarren