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

Merge main to live #3351

Merged
merged 5 commits into from
Nov 12, 2024
Merged

Merge main to live #3351

merged 5 commits into from
Nov 12, 2024

Conversation

zivkan
Copy link
Member

@zivkan zivkan commented Nov 7, 2024

No description provided.

rainersigwald and others added 4 commits October 9, 2024 15:45
Specifying the current value for the property is a best practice to avoid accidentally overwriting an existing value.
This doc should not reccomend that users create a `.` file.  That's a reserved directory alias for current directory.  I'd be surprised if that actually works, it would definitely confuse the build / extraction / etc.  I can't even create a file with this name.  The correct file name is `_._`.
@zivkan zivkan requested review from a team as code owners November 7, 2024 21:00
Copy link

Learn Build status updates of commit 7268a55:

💡 Validation status: suggestions

File Status Preview URL Details
docs/concepts/Auditing-Packages.md 💡Suggestion View Details
docs/reference/errors-and-warnings/NU1011.md ✅Succeeded View
docs/reference/nuspec.md ✅Succeeded View
docs/release-notes/NuGet-6.12.md ✅Succeeded View
docs/release-notes/NuGet-6.13.md ✅Succeeded View
docs/TOC.md ✅Succeeded View

docs/concepts/Auditing-Packages.md

  • Line 133, Column 7: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/nuget/consume-packages/Central-Package-Management#transitive-pinning' will be broken in isolated environments. Replace with a relative link.
  • Line 134, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/nuget/concepts/auditing-packages#excluding-advisories' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link

Learn Build status updates of commit 225c8c2:

✅ Validation status: passed

File Status Preview URL Details
docs/concepts/Auditing-Packages.md ✅Succeeded View
docs/reference/errors-and-warnings/NU1011.md ✅Succeeded View
docs/reference/nuspec.md ✅Succeeded View
docs/release-notes/NuGet-6.12.md ✅Succeeded View
docs/release-notes/NuGet-6.13.md ✅Succeeded View
docs/TOC.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@zivkan zivkan merged commit 386628b into live Nov 12, 2024
2 checks passed
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.

6 participants