Skip to content

Update NuGet.Config package sources#4521

Open
hlakudi200 wants to merge 2 commits intomainfrom
hlakudi200-patch-1
Open

Update NuGet.Config package sources#4521
hlakudi200 wants to merge 2 commits intomainfrom
hlakudi200-patch-1

Conversation

@hlakudi200
Copy link
Copy Markdown
Contributor

Updated shesha-core/.nuget/NuGet.Config to correct package source configuration for backend deployment.

Commented out legacy configuration block

Cleared existing package sources to avoid conflicts

Updated local-packages path to ../s/shesha-core/local-packages

Retained required NuGet v3, v2, and ImageSharp sources

Kept source control integration disabled

Updated NuGet.Config to define package sources and clear previous entries.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 16, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • shesha-core/.nuget/NuGet.Config is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hlakudi200-patch-1

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@IvanIlyichev IvanIlyichev left a comment

Choose a reason for hiding this comment

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

Hi @hlakudi200. Please do the following:

  1. remove commented out configuration, it doesn't make any sense to store this config
  2. relative path ../s/ is environment specific, if you really need it then it should be stored in a separate config file (e.g. NuGet.Azure.Config)
  3. Packages from local-packages folder are not used by the applicaiton, I believe you can safely remove it from config

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.

2 participants