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

Add Public NuGet Feed to Support Developers Without Private Feed Access #396

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

lei9444
Copy link
Contributor

@lei9444 lei9444 commented Dec 4, 2024

cherry-pick from 43a876e

Description

The original NuGet feed lacked public access support. To address this, a new public NuGet feed has been created under the shine-oss organization, utilizing the WinAppSDK Samples project. This feed now serves the WinAppSDK Samples repository, ensuring full open-source compliance and accessibility.

the Feed Link: https://dev.azure.com/shine-oss/WinAppSDK-Samples/_artifacts/feed/WinAppSDK-SampleDeps

Target Release

main branch

Checklist

  • Samples build and run using the Visual Studio versions listed in the Windows development docs.
  • Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
  • Samples set the minimum supported OS version to Windows 10 version 1809.
  • Samples build clean with no warnings or errors.
  • [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
  • If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
  • I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

…ss (#393)

* change feed

* add internal back

* update the order

* remove internal one

* update the template

* add comments back

* remove another one
@lei9444 lei9444 requested a review from alexlamtest December 4, 2024 01:51
@lei9444 lei9444 merged commit d5d0518 into main Dec 5, 2024
1 check passed
@lei9444 lei9444 deleted the user/leilzh/changefeed branch December 5, 2024 07:51
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