Skip to content

Allow pulling packages only from nuget.org. Add a build workflow to ensure the build is good for PRs and CI - #10

Merged
dixonjoel merged 7 commits into
mainfrom
users/jdixon/allow-nuget-org-only
Jun 29, 2026
Merged

dixonjoel merged 7 commits into
mainfrom
users/jdixon/allow-nuget-org-only

Conversation

@dixonjoel

Copy link
Copy Markdown
Contributor

What does this Pull Request accomplish?

Add a nuget.config file that makes sure only nuget.org is allowed when building in this repo.

Also add a workflow to ensure that the two .slnx projects build without error.

Why should this Pull Request be merged?

Make sure we are not depending on any internal dependencies from Artifactory and that we don't break the build for everyone using the repo with a user change.

What testing has been done?

PR should run the workflow or I'll run it manually with gh.

@dixonjoel
dixonjoel marked this pull request as ready for review June 18, 2026 16:36
Copilot AI review requested due to automatic review settings June 18, 2026 16:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR tightens NuGet package sourcing by adding a repo-level nuget.config that clears all feeds and allows only nuget.org, and adds a GitHub Actions workflow to build the repository’s two solution files on pushes and PRs.

Changes:

  • Added nuget.config to restrict package sources to https://api.nuget.org/v3/index.json (with source mapping).
  • Added a Windows GitHub Actions build workflow that runs dotnet build for examples/Examples.slnx and metadata-builder/MetadataBuilder.slnx.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
nuget.config Restricts NuGet restores to nuget.org only.
.github/workflows/build.yml Adds CI build validation for the two .slnx solutions on PRs and pushes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml
Comment thread .github/workflows/build.yml
dixonjoel and others added 3 commits June 22, 2026 13:04
@dixonjoel
dixonjoel requested a review from mjohanse-emr June 29, 2026 15:53
@dixonjoel

dixonjoel commented Jun 29, 2026 •

Copy link
Copy Markdown
Contributor Author

@mjohanse-emr I updated the README, so please take a look at that.

@dixonjoel
dixonjoel merged commit e510944 into main Jun 29, 2026
1 check passed
@dixonjoel
dixonjoel deleted the users/jdixon/allow-nuget-org-only branch June 29, 2026 19:57
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