Skip to content

Fix file header in src/Mvc#31430

Closed
Youssef1313 wants to merge 1 commit into
dotnet:mainfrom
Youssef1313:file-headers-mvc
Closed

Fix file header in src/Mvc#31430
Youssef1313 wants to merge 1 commit into
dotnet:mainfrom
Youssef1313:file-headers-mvc

Conversation

@Youssef1313
Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 commented Apr 1, 2021 •
edited by BrennanConroy
Loading

Part of #18873

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Apr 1, 2021
@mkArtakMSFT mkArtakMSFT added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Apr 1, 2021
@mkArtakMSFT mkArtakMSFT self-assigned this Apr 1, 2021
@mkArtakMSFT
Copy link
Copy Markdown
Contributor

mkArtakMSFT commented Apr 1, 2021

Thanks for your PR, @Youssef1313.
Unfortunately changes to the licensing related comments are things we have to do very carefully.
In fact, this is already in the works, so we're closing this PR.

Thanks for your effort!

@mkArtakMSFT mkArtakMSFT closed this Apr 1, 2021
@BrennanConroy
Copy link
Copy Markdown
Member

BrennanConroy commented Apr 1, 2021

This is in the works by @Youssef1313
See #30132

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mkArtakMSFT mkArtakMSFT

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Youssef1313 @mkArtakMSFT @BrennanConroy