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

Adding BuildAndPublishInsights.yml to CI.yml so that the headers and libs are included in the CI builds. #1149

Merged
merged 78 commits into from
Aug 7, 2021

Conversation

ZenBird-zz
Copy link
Contributor

No description provided.

@ghost ghost added the needs-triage label Jul 30, 2021
@ZenBird-zz ZenBird-zz requested review from MythiliM and kythant July 30, 2021 00:53
@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

zenbird and others added 21 commits August 3, 2021 11:59
* Adding a script to auto generate the rc file for the resource dll that will contain the values needed to log PartB telemetry. The PartB struct contains the SDK version, experimentation flag and other data
* The .rc file for the resource project is generated from the resource.templ file checked in with the project.
* The script must be executed before running the build step on this solution.
* Adding WindowsAppSDKTelemetry.h that will reference this resource dll to construct the PartB structure.
… project names (Microsoft.WindowsAppSDKTelemetry.lib)

* Added a property in Native.C.Props to reference the library when linking against WindowsAppSDK.Foundation nuget package
* Renamed the Insights lib project to conform to other WindowsAppSDK project names (Microsoft.WindowsAppSDK.Insights.lib)
* Added a property in Native.C.Props to reference the library when linking against WindowsAppSDK.Foundation nuget package
…nifest.xml build error (#1159)

* Adding an explicit reference to Microsoft.ApplicationModel.Resources.winmd causes invalid appxmanifest.xml build error

* Added resources winmd for C++/WinRT clients
* Adding PowerNotifications Proj

* Adding PowerNotifications Tests

* Adding FrameworkUDK Nuget

* Added entry to build/NuSpecs/AppxManifest
Added 'Factory()' to handle static functions
Removed TODO comments
{0} to {}

Co-authored-by: Huzaifa Danish <modanish@microsoft.com>, Brian Alker <brial@microsoft.com>
…s is so that only projects that reference the Insights will get a reference to Microsoft WIL.
…s is so that only projects that reference the Insights will get a reference to Microsoft WIL.
…s is so that only projects that reference the Insights will get a reference to Microsoft WIL.
@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ZenBird-zz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ZenBird-zz ZenBird-zz merged commit 7d88ac9 into main Aug 7, 2021
@ZenBird-zz ZenBird-zz deleted the user/zenbird/WindowsAppSDK branch August 7, 2021 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants