Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: pbrisbin/bugsnag-haskell
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.0.1.2
Choose a base ref
...
head repository: pbrisbin/bugsnag-haskell
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.0.1.3
Choose a head ref
  • 14 commits
  • 14 files changed
  • 1 contributor

Commits on Aug 3, 2018

  1. Add redactRequestHeaders

    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    e1c59f8 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a653a58 View commit details
    Browse the repository at this point in the history
  3. Brittany

    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    98bfe64 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    5cb2da9 View commit details
    Browse the repository at this point in the history
  5. Use bugsnagShouldNotify on the Event

    This is important so that any BeforeNotify behavior that changes aspects
    of the Event that would cause it to be ignored are respected. The
    trade-off is that we have to introduce an error call for the case of
    multiple Exceptions per Event. It's incoherent to pair bsIgnoreException
    with shouldNotify for a multi-Exception Event. Do we not-notify if any
    are ignored? All?
    
    In the future, we might want to either fully support multi-Exception
    Events by changing the setting to bsFilterExceptions (another
    BeforeNotify, which is nice), or fully *not* support it by changing the
    type of BugsnagEvent to hold a single Exception (not a NonEmpty). The
    former is a breaking change that will cascade throughout the library
    with little benefit (it would require making beExceptions a normal list
    and handling the empty case without the aide of types), and the latter
    would preclude using this library to interact with the API directly,
    because the types would disallow a valid use-case at that level. The
    current error-compromise might be our best option, even long-term.
    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    0d410c7 View commit details
    Browse the repository at this point in the history
  6. Revamp CI

    - Update LTS in default build
    - Test multiple versions of GHC
    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    66d04ca View commit details
    Browse the repository at this point in the history
  7. Add make watch

    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    0b7e6da View commit details
    Browse the repository at this point in the history
  8. Fix yesod example

    pbrisbin committed Aug 3, 2018
    Configuration menu
    Copy the full SHA
    adb1969 View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2018

  1. More comprehensive CI

    pbrisbin committed Aug 4, 2018
    Configuration menu
    Copy the full SHA
    c7b5d84 View commit details
    Browse the repository at this point in the history
  2. Set aeson version bounds

    We will rely on this minimum version to use a simpler import. It wasn't
    much trouble to bring in the extra-dep on old resolvers. If it were (or
    becomes that way in the future, or we get a user report about it), we
    can use CPP to support older aeson -- I'm not against it outright.
    pbrisbin committed Aug 4, 2018
    Configuration menu
    Copy the full SHA
    d39fce1 View commit details
    Browse the repository at this point in the history
  3. Avoid doctest in the 8.2 build

    Very strange error, not worth debugging for the not-latest build.
    pbrisbin committed Aug 4, 2018
    Configuration menu
    Copy the full SHA
    ab9d952 View commit details
    Browse the repository at this point in the history
  4. Avoiding linting on nightly

    Weeder can't find the correct Cabal file. Unclear why, the last time
    this was fixed by using STACK_YAML, which we're doing.
    pbrisbin committed Aug 4, 2018
    Configuration menu
    Copy the full SHA
    9e6b2e6 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    fd8b358 View commit details
    Browse the repository at this point in the history
  6. Version bump

    pbrisbin committed Aug 4, 2018
    Configuration menu
    Copy the full SHA
    b17e887 View commit details
    Browse the repository at this point in the history
Loading