Repository navigation
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Fifty-four sample links remain broken, and the redirect change removes coverage for an existing legacy URL.
1 open finding
What changed in this PR
Standardizes sample directories as samples so Dependabot patterns can cover them consistently.
Changes:
- Moves legacy sample directory variants into
samples. - Updates related snippet paths and selected metadata.
- Adjusts a legacy publishing redirect.
| File(s) | Description |
|---|---|
aspnetcore/**/samples/** |
Relocates sample projects and assets. |
| Related ASP.NET Core articles | Updates references to moved samples. |
.openpublishing.redirection.json |
Changes a legacy redirect source path. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| }, | ||
| { | ||
| "source_path": "aspnet/mvc/overview/getting-started/introduction/sample/code-location.md", | ||
| "source_path": "aspnet/mvc/overview/getting-started/introduction/samples/code-location.md", |
|
Is it ok if we wait to merge this until after I've cleared through most of the PRs? I don't want to create a large number of merge conflicts with open work. UPDATE: Yes, let's please hold off on this until at least next week. I just had to go through and fix a bunch of merge conflicts for the metadata updates. I'd like to get down to just a handful of open PRs, and then we can get this in smoothly. |
For the sake of the Dependabot config, let's use a standard samples directory name of "samples". (See #37822 (comment).)
Internal previews
Toggle expand/collapse
Note
This table shows the first 50 preview links (sorted alphabetically by file path) found in the OPS build report. For the full list, select OpenPublishing.Build Details within checks.
Build report