Skip to content

Added comments for improved readability of the sample - #400

Merged
Godly T.Alias (godlytalias) merged 1 commit into
release/experimentalfrom
user/godlyalias/backgroundtasksamples
Dec 12, 2024
Merged

Added comments for improved readability of the sample#400
Godly T.Alias (godlytalias) merged 1 commit into
release/experimentalfrom
user/godlyalias/backgroundtasksamples

Conversation

@godlytalias

@godlytalias Godly T.Alias (godlytalias) commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

Description

Added comments to improve readability of background task sample as per below feedback
microsoft/WindowsAppSDK#3840 (comment)

Target Release

1.7

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.

Signed-off-by: godlytalias <godlytalias@yahoo.co.in>
@godlytalias

Copy link
Copy Markdown
Contributor Author

/azp run SamplesCI-BackgroundTask

@godlytalias
Godly T.Alias (godlytalias) merged commit 2fec67d into release/experimental Dec 12, 2024
@godlytalias
Godly T.Alias (godlytalias) deleted the user/godlyalias/backgroundtasksamples branch December 12, 2024 16:18
Godly T.Alias (godlytalias) added a commit that referenced this pull request Apr 8, 2025
Signed-off-by: godlytalias <godlytalias@yahoo.co.in>
Godly T.Alias (godlytalias) added a commit that referenced this pull request Apr 8, 2025
* Added sample for WinAppSDK BackgroundTaskBuilder (#392)

* Added sample for WinAppSDK BackgroundTaskBuilder

Signed-off-by: Godly Alias <godlyalias@microsoft.com>

* Added comments for improved readability of the sample (#400)

Signed-off-by: godlytalias <godlytalias@yahoo.co.in>

* InProc BackgroundTask Samples (#413)

* Added InProc BackgroundTask Samples

Signed-off-by: godlytalias <godlytalias@yahoo.co.in>

* Support for trimming in BackgroundTask Samples (#421)

Signed-off-by: godlytalias <godlytalias@yahoo.co.in>

---------

Signed-off-by: godlytalias <godlytalias@yahoo.co.in>
shisan (qiutongMS) pushed a commit that referenced this pull request Sep 2, 2026
Bring the straightforward release/2.0-stable sample set into main in one reconciliation step: BackgroundTask, CameraCaptureUI, OAuth2Manager, Badge notifications, StoragePickers, Unpackaged WinForms, DrawingIsland, and UXFrameworksOnIslands.

Keep the stable sample behavior while adapting package references, native toolsets, documentation, and CI wiring to the current main infrastructure.

Stable evidence: PRs #367, #392, #395, #398, #400, #402, #403, #405, #413, #421, #422, #427, #455, #512, and #590.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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.

3 participants