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

Telemetry #381

Merged
merged 13 commits into from
Aug 26, 2015
Merged

Telemetry #381

merged 13 commits into from
Aug 26, 2015

Commits on Aug 5, 2015

  1. First set of change containing telemetry infrastructure and logging o…

    …f some datapoints.
    
    This provides infrastructure for for adding concrete implementations for telemetry logger which will implement ITelemetryFactory.
    nareshjo committed Aug 5, 2015
    Configuration menu
    Copy the full SHA
    f44fa19 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9be9c55 View commit details
    Browse the repository at this point in the history

Commits on Aug 10, 2015

  1. Making ITelemetryLogger disposable so that any logger whcih needs to …

    …do some cleanup e.g. flushing data can do it.
    
    Using $(VSTarget) token instead of hardcoded version numbers in telemetry project file
    nareshjo committed Aug 10, 2015
    Configuration menu
    Copy the full SHA
    56816dd View commit details
    Browse the repository at this point in the history
  2. Some simple fixes based on code review:

    - renamed DefaultTelemetryLogger and factory to DummyTelemetryLogger ...
    - some namespace sorting and tab/space alignment fixes
    - made classes for telemetry constants static
    nareshjo committed Aug 10, 2015
    Configuration menu
    Copy the full SHA
    f56f37a View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2015

  1. Applied most of the feedback from code review.

    - Removed all references to vslogger
    - Added some overrides in Dapapointcollection to add items more conviniently
    - some minor alignment/typo fixes
    - Made Telemetry project anycpu and removed unnecessary items from it. set comvisible false for it
    - Extracted code to set some common telemetry properties and logging of packageloaded event to a more common place so that it can be called from any package
    - removed logging of project open/create as it is already covered by vs telemetry
    nareshjo committed Aug 11, 2015
    Configuration menu
    Copy the full SHA
    b16a3a4 View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2015

  1. open source piece for ntvs telemetry, new design

    using new approach to copy and reference private telemetry dlls
    directly.
    Updating nuget.exe to latest as that is required by latest stable app
    insight nugets
    nareshjo committed Aug 24, 2015
    Configuration menu
    Copy the full SHA
    6dca996 View commit details
    Browse the repository at this point in the history

Commits on Aug 25, 2015

  1. Changes as per CR feedback

    Moved Telemetry events and properties classes to shared project and
    marked them internal
    Including app insights dlls in Dev11 and 12 only
    Updated telemetry dlls
    Fixed indentation
    some other minor fixes
    nareshjo committed Aug 25, 2015
    Configuration menu
    Copy the full SHA
    61cbdb2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    29709e9 View commit details
    Browse the repository at this point in the history

Commits on Aug 26, 2015

  1. Resolving merge conflicts

    nareshjo committed Aug 26, 2015
    Configuration menu
    Copy the full SHA
    19fb6a8 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    d4b14b6 View commit details
    Browse the repository at this point in the history
  3. Specifying telemetry dll as a binary to be dropped in unsignedbinarie…

    …s folder during build
    nareshjo committed Aug 26, 2015
    Configuration menu
    Copy the full SHA
    c24868e View commit details
    Browse the repository at this point in the history
  4. Changing to OutputBinaries to OutputBinariesToSign to be consistent w…

    …ith a parallel PR and avoid conflicts later
    nareshjo committed Aug 26, 2015
    Configuration menu
    Copy the full SHA
    1b57cc7 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    ed3c4e7 View commit details
    Browse the repository at this point in the history