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

Chore: refactor reporting logic (refs #9161) #9168

Merged
merged 5 commits into from
Aug 30, 2017
Merged

Chore: refactor reporting logic (refs #9161) #9168

merged 5 commits into from
Aug 30, 2017

Commits on Aug 28, 2017

  1. Chore: refactor reporting logic (refs #9161)

    @not-an-aardvark
    not-an-aardvark committed Aug 28, 2017
    Configuration menu
    Copy the full SHA
    015d843 View commit details
    Browse the repository at this point in the history

Commits on Aug 29, 2017

  1. Ensure id is passed to rule contexts

    @not-an-aardvark
    not-an-aardvark committed Aug 29, 2017
    Configuration menu
    Copy the full SHA
    636067f View commit details
    Browse the repository at this point in the history

Commits on Aug 30, 2017

  1. Improve verify() performance

    @not-an-aardvark
    not-an-aardvark committed Aug 30, 2017
    Configuration menu
    Copy the full SHA
    f516143 View commit details
    Browse the repository at this point in the history

  2. Fix incorrect tests

    @not-an-aardvark
    not-an-aardvark committed Aug 30, 2017
    Configuration menu
    Copy the full SHA
    600bda3 View commit details
    Browse the repository at this point in the history

  3. Fix outdated JSDoc comment 

    While investigating the performance impact of this change, I tried modifying the `createReportTranslator` API, but I forgot to change the JSDoc comment back after reverting that modification.
    @not-an-aardvark
    not-an-aardvark committed Aug 30, 2017
    Configuration menu
    Copy the full SHA
    6973223 View commit details
    Browse the repository at this point in the history