Skip to content

Wire up console.warn/console.error to Raven.captureMessage #85

Closed

Description

@mattrobenolt

Explicitly monkey patch console.* as log levels.

Activity

  1. defunctzombie commented on Mar 14, 2013

    @defunctzombie
    Contributor

    No.

  2. acdha commented on Mar 14, 2013

    @acdha
    Contributor

    I'm -0 on this - seems like it'd be noisy and only helps developers who'd be in a position to do it the right way instead.

  3. mattrobenolt commented on Mar 14, 2013

    @mattrobenolt
    ContributorAuthor

    @acdha , it would be behind a log level setting. So you'd say, Raven.config(..., {level: 'warn'}) so you can control how much noise you want.

  4. acdha commented on Mar 14, 2013

    @acdha
    Contributor

    I wouldn't object to you adding it but I do wonder whether it would just encourage people to send poorly crafted log messages rather than integrate logging correctly.

  5. bisrael commented on Mar 15, 2013

    @bisrael
    Contributor

    usually when you do a Raven.captureMessage or captureException, you could put a console.log there if you like, or write a small wrapper for yourself, would be like two lines of code for the developer.

    Edit: Really the developer should be wrapping calls to Raven.capture* anyways, otherwise you're really coupling yourself to the Raven implementation.

  6. mattrobenolt commented on Oct 21, 2013

    @mattrobenolt
    ContributorAuthor

    This is done with the console plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions