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.

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

Clarify NU1105 #1923

Merged
merged 2 commits into from
Feb 28, 2020
Merged

Clarify NU1105 #1923

merged 2 commits into from
Feb 28, 2020

Conversation

nkolev92
Copy link
Member

@karann-msft
Copy link
Contributor

Docs Build status updates of commit 6ba6991:

✅ Validation status: passed

File Status Preview URL Details
docs/reference/errors-and-warnings/NU1105.md ✅Succeeded View

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Copy link
Member

@zivkan zivkan left a comment

Choose a reason for hiding this comment

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

I've heard podcasts where people starting in the industry say that hearing or reading words like "just", "simply", "obviously", and other similar words, make them think they might not be cut out for the industry when the information was new to them. We should avoid phrasing sentences in ways that could be interpreted as implying some knowledge should already be known.

docs/reference/errors-and-warnings/NU1105.md Outdated Show resolved Hide resolved
Co-Authored-By: Andy Zivkovic <zivkan@users.noreply.github.com>
@karann-msft
Copy link
Contributor

Docs Build status updates of commit 38c4089:

✅ Validation status: passed

File Status Preview URL Details
docs/reference/errors-and-warnings/NU1105.md ✅Succeeded View

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@joelverhagen joelverhagen deleted the master-nkolev92-NU1105 branch April 19, 2021 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

What is the “after imports” target?
4 participants