Skip to content

Remove broken Shadi Alnamrouti by-line link from dependency injection docs#36834

Merged
wadepickett merged 2 commits intomainfrom
copilot/remove-broken-by-line-link
Mar 4, 2026
Merged

Remove broken Shadi Alnamrouti by-line link from dependency injection docs#36834
wadepickett merged 2 commits intomainfrom
copilot/remove-broken-by-line-link

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 4, 2026

The Stack Overflow profile link for Shadi Alnamrouti no longer exists and appears broken in the dependency injection controller docs. Removes the link from both affected files and cleans up surrounding content.

Changes

  • aspnetcore/mvc/controllers/dependency-injection.md — removed broken author link and orphaned <!-- @shadialnamrouti --> comment from the >= aspnetcore-8.0 moniker section; updated ms.date
  • aspnetcore/mvc/controllers/includes/dependency-injection7.md — removed broken author link from the aspnetcore-3.0–8.0 moniker section; remaining two authors re-joined with "and"
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove broken by line link throughout repo</issue_title>
<issue_description>### Description

This by line link no longer exists. Remove from this topic and remove throughout the repo:
Shadi Alnamrouti

Page URL

https://learn.microsoft.com/en-us/aspnet/core/mvc/controllers/dependency-injection?view=aspnetcore-10.0&source=docs

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/mvc/controllers/dependency-injection.md

Document ID

f9a790d8-f25b-0e1d-eb63-c8ed3ac518cf

Platform Id

f13329f0-75ef-9f97-02c1-0df7ed49f582

Article author

@ardalis

Metadata

  • ID: 08d2cf2e-55fb-33a4-b1d4-c97b2d531741
  • PlatformId: f13329f0-75ef-9f97-02c1-0df7ed49f582
  • Service: aspnet-core
  • Sub-service: mvc

Related Issues</issue_description>

<agent_instructions>Be sure to check entire repo for this link to be removed. Make sure the remaining content after the link is removed is edited to make sense and follow guidelines.</agent_instructions>

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


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Internal previews

📄 File 🔗 Preview link
aspnetcore/mvc/controllers/dependency-injection.md aspnetcore/mvc/controllers/dependency-injection

… docs

Co-authored-by: wadepickett <10985336+wadepickett@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove broken by line link throughout repo Remove broken Shadi Alnamrouti by-line link from dependency injection docs Mar 4, 2026
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

Approved.

@wadepickett wadepickett marked this pull request as ready for review March 4, 2026 18:45
@wadepickett wadepickett self-requested a review March 4, 2026 18:45
@wadepickett wadepickett merged commit 536ac29 into main Mar 4, 2026
4 checks passed
@wadepickett wadepickett deleted the copilot/remove-broken-by-line-link branch March 4, 2026 18:46
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.

Remove broken by line link throughout repo

3 participants