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

Update NU1100.md #2829

Merged
merged 1 commit into from
Aug 3, 2022
Merged

Update NU1100.md #2829

merged 1 commit into from
Aug 3, 2022

Conversation

waldyd
Copy link
Contributor

@waldyd waldyd commented Jul 28, 2022

NU1100 may also be caused by the lack of a NuGet configuration file

NU1100 may also be caused by the lack of a NuGet configura
@opbld32
Copy link

opbld32 commented Jul 28, 2022

Docs Build status updates of commit b111b96:

✅ Validation status: passed

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

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@JonDouglas JonDouglas left a comment

Choose a reason for hiding this comment

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

Thank you kindly for the contribution!

A dependency could not be resolved. This is a generic issue for types that are not packages or projects.
A dependency could not be resolved. This is a generic issue for types that are not packages or projects.

This issue may also be caused by the lack of a NuGet configuration file.
Copy link
Member

Choose a reason for hiding this comment

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

NuGet would normally be creating the user-wide configuration file if it doesn't already exist.

Can you elaborate how you hit this?

@waldyd
Copy link
Contributor Author

waldyd commented Aug 3, 2022 via email

@JonDouglas JonDouglas merged commit bc38abb into NuGet:main Aug 3, 2022
@nkolev92
Copy link
Member

nkolev92 commented Aug 3, 2022

@JonDouglas I think we merged prematurely here.

I think we need to confirm this repro first, as it doesn't really make sense to me immediately.

@JonDouglas
Copy link
Contributor

@nkolev92 A simple reproduction is to remove the contents of a nuget.config file as if it were "empty". i.e. leave only the xml & configuration tags.

I repro'd it and the contribution here is helpful as the first scenario asks to check if dependencies exist on package sources, but the reality is the config may be generated by other tooling causing issues and one may want to check the config for accuracy.

This is similar to the problem of other nuget-powered tooling generating an empty config and the user has no idea how to resolve it.

@nkolev92
Copy link
Member

nkolev92 commented Aug 3, 2022

A simple reproduction is to remove the contents of a nuget.config file as if it were "empty". i.e. leave only the xml & configuration tags.

That's not what the update says. It talks about a missing nuget.config, which is different.

@JonDouglas
Copy link
Contributor

JonDouglas commented Aug 3, 2022

@nkolev92 The assumption here is that there exists no config in the project/solution and the error is thrown because the user-level is problematic (i.e. empty from other tooling or similar).

using the dotnet new nugetconfig generates a proper configuration at the project level that will override the user-level and thus works.

Perhaps the language we want to instill here is fixing these items at the user-level or inspecting the various configs to ensure the sources exist in the first place?

I still think the contribution is fine as is, but we can improve more later.

That is my understanding and reproduction of the issue however!

@nkolev92
Copy link
Member

nkolev92 commented Aug 3, 2022

I don't think we can say missing config because a missing config altogether causes NuGet to create one.
It is a critical onboarding experience.

The context with which it gets created is irrelevant imo.
The docs shouldn't contradict the actual tool behavior even if it's not the most user friendly description.

Perhaps the language we want to instill here is fixing these items at the user-level or inspecting the various configs to ensure the sources exist in the first place?

I'll take a crack at updating this and create a PR. I think it'll be easier to make progress that way :D

@JonDouglas
Copy link
Contributor

@waldyd As one last question, can you let us know the contents of your user config?

https://docs.microsoft.com/nuget/consume-packages/configuring-nuget-behavior#config-file-locations-and-uses

@nkolev92 nkolev92 mentioned this pull request Aug 3, 2022
@nkolev92
Copy link
Member

nkolev92 commented Aug 3, 2022

Hoping the following version improves clarity.

#2838.

@waldyd
Copy link
Contributor Author

waldyd commented Aug 3, 2022 via email

@nkolev92
Copy link
Member

nkolev92 commented Aug 3, 2022

NuGet creates a settings file if there's none.
That's the onboarding experience when you install new tools. There's no other component that creates a config file by default other than NuGet.

cc @v-luzh

Can you please try to repro this?

Start off with a new windows machine and follow the steps here:

#2829 (comment)

JonDouglas added a commit that referenced this pull request Aug 9, 2022
* document The parameter is incorrect behavior (#2800)

* document The parameter is incorrect behavior

* Add a note/tip instead of bold.

Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Add a dedicated page for props and targets (#2807)

* Document NU1802 and NU1803 (#2816)

* Add list in Normalized version numbers (#2818)

Fix #2817

* Clarify headline of section about projects without any package dependencies (#2812)

* Add issue management bots (#2825)

* Fix formatting typo in CPM (#2832)

* Add Gitea to NuGet hosting solutions (#2828)

* Add Gitea to NuGet hosting solutions

* Use Gitea project URL

* Update NU1100.md (#2829)

NU1100 may also be caused by the lack of a NuGet configura

* Updates to ms.date values based on file history (#2831)

* Updates to ms.date values based on file history

* edit

* fixed year

* Alphabetize nuget list (#2837)

* Improve NU1100  (#2838)

* Add CODEOWNERS to docs PRs (#2836)

* Release notes for 6.3 (#2834)

* release notes for 6.3

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: andrei <andreiepure@users.noreply.github.com>
Co-authored-by: riQQ <riQQ@users.noreply.github.com>
Co-authored-by: Drew Noakes <git@drewnoakes.com>
Co-authored-by: John Olheiser <john+github@jolheiser.com>
Co-authored-by: Nestor Waldyd <nwaldyd@yahoo.co.uk>
Co-authored-by: Mike Jones <mikejo@microsoft.com>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Kartheek Penagamuri <52756182+kartheekp-ms@users.noreply.github.com>
Co-authored-by: Martin Ruiz <martin.ruiz.mares@gmail.com>
Mikejo5000 added a commit that referenced this pull request Jun 19, 2023
* Change link for pre-release guidance

* Update release notes images

* End of May 2022 Live Update (#2763)

* June 2022 docs merge (#2783)

* Add 5.9 release notes (#2780)

* Add 6.0.2 release notes (#2774)

* Add 5.11 release notes (#2775)

* Add 5.7 release notes (#2776)

* Add 4.9 release notes (#2777)

* Add 5.2 release notes (#2778)

* Merge main to live (#2808)

* document The parameter is incorrect behavior (#2800)

* document The parameter is incorrect behavior

* Add a note/tip instead of bold.

Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Add a dedicated page for props and targets (#2807)

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Updates to ms.date values based on file history

* edit

* fixed year

* August Docs Live Merge (#2840)

* document The parameter is incorrect behavior (#2800)

* document The parameter is incorrect behavior

* Add a note/tip instead of bold.

Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Add a dedicated page for props and targets (#2807)

* Document NU1802 and NU1803 (#2816)

* Add list in Normalized version numbers (#2818)

Fix #2817

* Clarify headline of section about projects without any package dependencies (#2812)

* Add issue management bots (#2825)

* Fix formatting typo in CPM (#2832)

* Add Gitea to NuGet hosting solutions (#2828)

* Add Gitea to NuGet hosting solutions

* Use Gitea project URL

* Update NU1100.md (#2829)

NU1100 may also be caused by the lack of a NuGet configura

* Updates to ms.date values based on file history (#2831)

* Updates to ms.date values based on file history

* edit

* fixed year

* Alphabetize nuget list (#2837)

* Improve NU1100  (#2838)

* Add CODEOWNERS to docs PRs (#2836)

* Release notes for 6.3 (#2834)

* release notes for 6.3

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: andrei <andreiepure@users.noreply.github.com>
Co-authored-by: riQQ <riQQ@users.noreply.github.com>
Co-authored-by: Drew Noakes <git@drewnoakes.com>
Co-authored-by: John Olheiser <john+github@jolheiser.com>
Co-authored-by: Nestor Waldyd <nwaldyd@yahoo.co.uk>
Co-authored-by: Mike Jones <mikejo@microsoft.com>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Kartheek Penagamuri <52756182+kartheekp-ms@users.noreply.github.com>
Co-authored-by: Martin Ruiz <martin.ruiz.mares@gmail.com>

* November / NuGet 6.4 Merge (#2934)

* In all content, remove (#2920)

ms.prod = nuget
Replace with
ms.service = nuget

Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>

* Update VS extensibility docs (#2929)

* Document various log codes (#2933)

Documenting: NU1008, NU1009, NU1010, NU1011, NU1109, NU1402, NU1403, NU5042, NU5049

* Release notes for 6.4 (#2926)

* Suggest a more generally appropriate import tag (#2932)

* Suggest a more generally appropriate import tag

Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>
Co-authored-by: Andy Zivkovic <zivkan@users.noreply.github.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: Jean-Pierre Briedé <jebriede@microsoft.com>
Co-authored-by: Andrew Arnott <andrewarnott@gmail.com>

* Ms.date update

---------

Co-authored-by: Christopher R. Gill <christopher.gill@microsoft.com>
Co-authored-by: Jon Douglas <jodou@microsoft.com>
Co-authored-by: Christopher R. Gill <chrisraygill@gmail.com>
Co-authored-by: Joel Verhagen <jver@microsoft.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>
Co-authored-by: Kartheek Penagamuri <52756182+kartheekp-ms@users.noreply.github.com>
Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: andrei <andreiepure@users.noreply.github.com>
Co-authored-by: riQQ <riQQ@users.noreply.github.com>
Co-authored-by: Drew Noakes <git@drewnoakes.com>
Co-authored-by: John Olheiser <john+github@jolheiser.com>
Co-authored-by: Nestor Waldyd <nwaldyd@yahoo.co.uk>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Martin Ruiz <martin.ruiz.mares@gmail.com>
Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>
Co-authored-by: Andy Zivkovic <zivkan@users.noreply.github.com>
Co-authored-by: Jean-Pierre Briedé <jebriede@microsoft.com>
Co-authored-by: Andrew Arnott <andrewarnott@gmail.com>
Co-authored-by: Your Name <you@example.com>
Mikejo5000 added a commit that referenced this pull request Aug 21, 2023
* Change link for pre-release guidance

* Update release notes images

* End of May 2022 Live Update (#2763)

* June 2022 docs merge (#2783)

* Add 5.9 release notes (#2780)

* Add 6.0.2 release notes (#2774)

* Add 5.11 release notes (#2775)

* Add 5.7 release notes (#2776)

* Add 4.9 release notes (#2777)

* Add 5.2 release notes (#2778)

* Merge main to live (#2808)

* document The parameter is incorrect behavior (#2800)

* document The parameter is incorrect behavior

* Add a note/tip instead of bold.

Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Add a dedicated page for props and targets (#2807)

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Updates to ms.date values based on file history

* edit

* fixed year

* August Docs Live Merge (#2840)

* document The parameter is incorrect behavior (#2800)

* document The parameter is incorrect behavior

* Add a note/tip instead of bold.

Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>

* Add a dedicated page for props and targets (#2807)

* Document NU1802 and NU1803 (#2816)

* Add list in Normalized version numbers (#2818)

Fix #2817

* Clarify headline of section about projects without any package dependencies (#2812)

* Add issue management bots (#2825)

* Fix formatting typo in CPM (#2832)

* Add Gitea to NuGet hosting solutions (#2828)

* Add Gitea to NuGet hosting solutions

* Use Gitea project URL

* Update NU1100.md (#2829)

NU1100 may also be caused by the lack of a NuGet configura

* Updates to ms.date values based on file history (#2831)

* Updates to ms.date values based on file history

* edit

* fixed year

* Alphabetize nuget list (#2837)

* Improve NU1100  (#2838)

* Add CODEOWNERS to docs PRs (#2836)

* Release notes for 6.3 (#2834)

* release notes for 6.3

Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: andrei <andreiepure@users.noreply.github.com>
Co-authored-by: riQQ <riQQ@users.noreply.github.com>
Co-authored-by: Drew Noakes <git@drewnoakes.com>
Co-authored-by: John Olheiser <john+github@jolheiser.com>
Co-authored-by: Nestor Waldyd <nwaldyd@yahoo.co.uk>
Co-authored-by: Mike Jones <mikejo@microsoft.com>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Kartheek Penagamuri <52756182+kartheekp-ms@users.noreply.github.com>
Co-authored-by: Martin Ruiz <martin.ruiz.mares@gmail.com>

* November / NuGet 6.4 Merge (#2934)

* In all content, remove (#2920)

ms.prod = nuget
Replace with
ms.service = nuget

Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>

* Update VS extensibility docs (#2929)

* Document various log codes (#2933)

Documenting: NU1008, NU1009, NU1010, NU1011, NU1109, NU1402, NU1403, NU5042, NU5049

* Release notes for 6.4 (#2926)

* Suggest a more generally appropriate import tag (#2932)

* Suggest a more generally appropriate import tag

Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>
Co-authored-by: Andy Zivkovic <zivkan@users.noreply.github.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: Jean-Pierre Briedé <jebriede@microsoft.com>
Co-authored-by: Andrew Arnott <andrewarnott@gmail.com>

* Ms.date update

* Updating ms.date

---------

Co-authored-by: Christopher R. Gill <christopher.gill@microsoft.com>
Co-authored-by: Jon Douglas <jodou@microsoft.com>
Co-authored-by: Christopher R. Gill <chrisraygill@gmail.com>
Co-authored-by: Joel Verhagen <jver@microsoft.com>
Co-authored-by: Nikolche Kolev <nikolce1kolev@gmail.com>
Co-authored-by: Jon Douglas <jonathan.douglas@microsoft.com>
Co-authored-by: Kartheek Penagamuri <52756182+kartheekp-ms@users.noreply.github.com>
Co-authored-by: Simon Cropp <simon.cropp@gmail.com>
Co-authored-by: andrei <andreiepure@users.noreply.github.com>
Co-authored-by: riQQ <riQQ@users.noreply.github.com>
Co-authored-by: Drew Noakes <git@drewnoakes.com>
Co-authored-by: John Olheiser <john+github@jolheiser.com>
Co-authored-by: Nestor Waldyd <nwaldyd@yahoo.co.uk>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Martin Ruiz <martin.ruiz.mares@gmail.com>
Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Docs Allowlist Management <docs-allowlist-mgmt@microsoft.com>
Co-authored-by: Andy Zivkovic <zivkan@users.noreply.github.com>
Co-authored-by: Jean-Pierre Briedé <jebriede@microsoft.com>
Co-authored-by: Andrew Arnott <andrewarnott@gmail.com>
Co-authored-by: Your Name <you@example.com>
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.

4 participants