Skip to content
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

Remove fixing-warnings.md since it is duplicated in dotnet/docs #2645

Merged
merged 2 commits into from
Mar 8, 2022
Merged

Remove fixing-warnings.md since it is duplicated in dotnet/docs #2645

merged 2 commits into from
Mar 8, 2022

Conversation

jtschuster
Copy link
Member

No description provided.

@jtschuster jtschuster requested review from vitek-karas and agocke March 1, 2022 22:19
vitek-karas
vitek-karas approved these changes Mar 2, 2022
View reviewed changes
Copy link
Member

@vitek-karas vitek-karas left a comment

Choose a reason for hiding this comment

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

I think it might be a good idea to add links to the official docs to the readme.md in the docs directory - so that if somebody find their way to the repo, they can find the official docs easily.

@jtschuster jtschuster merged commit e003f2c into dotnet:main Mar 8, 2022
agocke pushed a commit to dotnet/runtime that referenced this pull request Nov 16, 2022
@jtschuster
Remove fixing-warnings.md and link to dotnet/docs (dotnet/linker#2645)
501ebec 
Commit migrated from dotnet/linker@e003f2c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vitek-karas vitek-karas vitek-karas approved these changes

@marek-safar marek-safar Awaiting requested review from marek-safar

@agocke agocke Awaiting requested review from agocke

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jtschuster @vitek-karas