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.

Sign up for GitHub

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

Jump to bottom

Suppress obsolete warning for TelemetrySource (temporarily) #3381

Closed
wants to merge 1 commit into from
Closed

Suppress obsolete warning for TelemetrySource (temporarily) #3381

wants to merge 1 commit into from

Conversation

anpete
Copy link
Contributor

@anpete anpete commented Oct 8, 2015

No description provided.

@anpete
Copy link
Contributor Author

anpete commented Oct 8, 2015

cc @bricelam in case I did this incorrectly 🍤

@bricelam
Copy link
Contributor

bricelam commented Oct 8, 2015

Hmm, I don't see an option for noWarn; this may not be supported in project.json

@bricelam
Copy link
Contributor

bricelam commented Oct 8, 2015

...but we get three suppressed for free.

@bricelam
Copy link
Contributor

bricelam commented Oct 8, 2015

Ah, we want aspnet/dnx#2042

@bricelam
Copy link
Contributor

bricelam commented Oct 8, 2015

:shipit: (can remove them from project.json if you want)

@anpete
Copy link
Contributor Author

anpete commented Oct 8, 2015

Thanks @bricelam will use #pragma I guess.

@anpete anpete closed this Oct 8, 2015
@anpete anpete deleted the build-warning branch October 8, 2015 23:20
@Eilon
Copy link
Member

Eilon commented Oct 9, 2015

Do we have a bug to track moving to the new DiagnosticsSource and removing the #pragmas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@anpete @bricelam @Eilon @dnfclas