WindowsAppSDK icon indicating copy to clipboard operation
WindowsAppSDK copied to clipboard
verified publisher icon microsoft

Address argument encoding nits

Open pmpurifoy opened this issue 3 years ago • 16 comments

This PR addresses the nits listed in from the previously closed PR: https://github.com/microsoft/WindowsAppSDK/pull/2805.

Add sample to display format: https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938203444

Can we return empty string in value for arguments? https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938209869

Tokenizing: https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938213694

Removing unnecessary c_strs: https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938219862 https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938241055

pmpurifoy avatar Aug 05 '22 00:08 pmpurifoy

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 05 '22 01:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 05 '22 01:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 05 '22 01:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 05 '22 01:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 05 '22 01:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 05 '22 01:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 05 '22 03:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 05 '22 03:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 05 '22 17:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 05 '22 17:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 06 '22 00:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 06 '22 00:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 06 '22 03:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 06 '22 03:08 azure-pipelines[bot]

/azp run TransportPackage-Foundation-PR

loneursid avatar Aug 08 '22 20:08 loneursid

No pipelines are associated with this pull request.

azure-pipelines[bot] avatar Aug 08 '22 20:08 azure-pipelines[bot]

Closing PRs that appear to have been abandoned.

bpulliam avatar Apr 03 '23 12:04 bpulliam