Skip to content

Always enable telemetry#6

Merged
AdamYoblick merged 12 commits intomasterfrom
yamlfixes
Oct 25, 2018
Merged

Always enable telemetry#6
AdamYoblick merged 12 commits intomasterfrom
yamlfixes

Conversation

@AdamYoblick
Copy link
Contributor

No description provided.

variables:
Build.Repository.Clean: true
_enableTelemetry: true
_HelixType: build/product
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a little confusing because I removed this line. It's because the eng/build.yml template has enableTelemetry as a parameter and it defaults to true if nothing is passed in. So I pass in nothing.

@AdamYoblick AdamYoblick merged commit 3cf9c23 into master Oct 25, 2018
@AdamYoblick AdamYoblick deleted the yamlfixes branch October 25, 2018 22:21
zsd4yr pushed a commit that referenced this pull request Nov 26, 2018
# This is the 1st commit message:

fixes for DpiHelper to use TryFindDpiAwarenessContextsEqual, fixes to CommonUnsafeNativeMethods to check for unique isntance of enum before performing work with extern

# This is the commit message #2:

moving these files over here

# This is the commit message #3:

some more tests

# This is the commit message #4:

some more tests and more fixes

# This is the commit message #5:

left some merge conflict resoluts. behind

# This is the commit message #6:

Update CommonUnsafeNativeMethods.cs
# This is the commit message #7:

Update CommonUnsafenativeMethods.cs
# This is the commit message #8:

Update DpiHelper.cs
@ghost ghost locked as resolved and limited conversation to collaborators Feb 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant