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

Switch to CS2.0 core library #2

Closed
wants to merge 1 commit into from
Closed

Conversation

southwood
Copy link

This is the core library for the android project, it compiles in Java1.6 and can send CS2.0 telemetry. To use it a platform specific context must be implemented (implement ITelemetryContext) and CoreTelemetryClient must be extended to instantiate the context.

This is the core library for the android project, it compiles in Java1.6 and can send CS2.0 telemetry. To use it a platform specific context must be implemented (implement ITelemetryContext) and CoreTelemetryClient must be extended to instantiate the context.
@@ -0,0 +1,13 @@
package com.microsoft.applicationinsights.channel;

public interface IChannelConfig {
Copy link
Contributor

Choose a reason for hiding this comment

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

So we're going with the 'I' prefix for interfaces?

Copy link
Author

Choose a reason for hiding this comment

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

yes

@littleaj
Copy link
Contributor

This pull request is very old and is being pruned. The branch still exists if someone thinks this should be reopened.

@littleaj littleaj closed this Nov 27, 2017
@dhaval24 dhaval24 deleted the features/update-to-cs20 branch December 2, 2017 00:55
littleaj pushed a commit that referenced this pull request Jan 4, 2018
trask pushed a commit that referenced this pull request Nov 7, 2020
* apache camel 2.20.x instrumentation

* removed instrumentation advice from helper classes

* code review #1

* code review #2 / experimental server name update

* code review changes / new tests, improved direct, new license header

* rebase changes

* code review changes

* code review changes, REST tests added

* changes after rebase to newest master

* code review changes

* code review

* code review

* rebase to master
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