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

Add new visitor to get deterministic SARIF log by sorting results #2422

Merged
merged 29 commits into from
Mar 2, 2022
Merged

Add new visitor to get deterministic SARIF log by sorting results #2422

merged 29 commits into from
Mar 2, 2022

Commits on Jan 17, 2022

  1. Add new visitor to get deterministic SARIF log by sorting results

    @yongyan-gh
    yongyan-gh committed Jan 17, 2022
    Configuration menu
    Copy the full SHA
    22ec32b View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh authored Jan 17, 2022
    Configuration menu
    Copy the full SHA
    44a9f46 View commit details
    Browse the repository at this point in the history

  3. Fix dotnet format issue

    @yongyan-gh
    yongyan-gh committed Jan 17, 2022
    Configuration menu
    Copy the full SHA
    3c597ff View commit details
    Browse the repository at this point in the history

Commits on Jan 18, 2022

  1. updating format

    @yongyan-gh
    yongyan-gh committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    ca5c356 View commit details
    Browse the repository at this point in the history

Commits on Jan 20, 2022

  1. remove unnecessary using 

    Format & minor fixes
    @yongyan-gh
    yongyan-gh committed Jan 20, 2022
    Configuration menu
    Copy the full SHA
    043079f View commit details
    Browse the repository at this point in the history

Commits on Jan 21, 2022

  1. Add Run Comparer to support sorting logs with multiple runs.

    @yongyan-gh
    yongyan-gh committed Jan 21, 2022
    Configuration menu
    Copy the full SHA
    ea82e82 View commit details
    Browse the repository at this point in the history

Commits on Jan 27, 2022

  1. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh authored Jan 27, 2022
    Configuration menu
    Copy the full SHA
    3961924 View commit details
    Browse the repository at this point in the history

  2. Add command argument unit tests 

    fix dotnet format
    @yongyan-gh
    yongyan-gh committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    d568f85 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'users/yongyan-gh/sortingvisitor' of https://github.com/… 

    …microsoft/sarif-sdk into users/yongyan-gh/sortingvisitor
    @yongyan-gh
    yongyan-gh committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    07e1b4c View commit details
    Browse the repository at this point in the history

Commits on Feb 2, 2022

  1. use ContainsKey to avoid allocating variable

    @yongyan-gh
    yongyan-gh committed Feb 2, 2022
    Configuration menu
    Copy the full SHA
    cfebb85 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2022

  1. Fixing AnalyzeCommandBase and MultithreadedAnalyzeCommandBase art… 

    …ifacts generation (#2433)

* Fixing multithreaded artifacts generation

* Fixing tests and flags

* Loosing precision.

* Applying fix for AnalyzeCommandBase

* Enabling tests

* Updating test case and release history

* Creating const to prevent magical numbers everywhere

* Rebaselining tests

* Creating Artifacts flag to keep previous behavior

* Addressing PR feedback.

* Rollback changes

* Update SARIF2012.ProvideRuleProperties_Invalid.sarif

* updating back

* Ordering deprecated names
    @eddynaka @yongyan-gh
    eddynaka authored and yongyan-gh committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    351ef43 View commit details
    Browse the repository at this point in the history

  2. SarifLogger now emits an artifacts table entry if `artifactLocation… 

    …` is not null for tool configuration and tool execution notifications. (#2437)

* Fixing artifacts generation when logging notifications

* Updating release history.

* Updating ReleaseHistory
    @eddynaka @yongyan-gh
    eddynaka authored and yongyan-gh committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    9f140e4 View commit details
    Browse the repository at this point in the history

  3. Fix ArgumentException when recurse is enabled and two file target s… 

    …pecifiers generates the same file paths (#2438)

* Fixing ArgumentException when passing two filePaths that generates duplicated file analysis

* Fixing dotnet-format issues and updating releasehistory

* Removing comments

* Addressing PR feedback

* Addressing PR feedback
    @eddynaka @yongyan-gh
    eddynaka authored and yongyan-gh committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    0727702 View commit details
    Browse the repository at this point in the history

  4. Addressing PR review issues 

    Add suppression support (#2435)

* Add suppression support

* Add incompatibility check and make suppressions non-null

Co-authored-by: Eddy Nakamura <eddynaka@gmail.com>

Update releasehistory

fix couple test cases
    @yongyan-gh
    yongyan-gh committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    6306faf View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2022

  1. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh authored Feb 9, 2022
    Configuration menu
    Copy the full SHA
    f192c36 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' of https://github.com/microsoft/sarif-sdk

    @yongyan-gh
    yongyan-gh committed Feb 9, 2022
    Configuration menu
    Copy the full SHA
    bb3ce50 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh committed Feb 9, 2022
    Configuration menu
    Copy the full SHA
    196cc0a View commit details
    Browse the repository at this point in the history

Commits on Feb 11, 2022

  1. Fix issues in PR review

    @yongyan-gh
    yongyan-gh committed Feb 11, 2022
    Configuration menu
    Copy the full SHA
    9fa0ca7 View commit details
    Browse the repository at this point in the history

  2. Add xml comments

    @yongyan-gh
    yongyan-gh committed Feb 11, 2022
    Configuration menu
    Copy the full SHA
    9d1ce42 View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2022

  1. Fix test issues

    @yongyan-gh
    yongyan-gh committed Feb 14, 2022
    Configuration menu
    Copy the full SHA
    305570e View commit details
    Browse the repository at this point in the history

  2. fix dotnet format

    @yongyan-gh
    yongyan-gh committed Feb 14, 2022
    Configuration menu
    Copy the full SHA
    4747ec1 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh authored Feb 14, 2022
    Configuration menu
    Copy the full SHA
    9281b44 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2022

  1. Addressing review feedbacks

    @yongyan-gh
    yongyan-gh committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    314bd53 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh authored Feb 18, 2022
    Configuration menu
    Copy the full SHA
    d37dd18 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @yongyan-gh
    yongyan-gh committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    eb150f7 View commit details
    Browse the repository at this point in the history

  4. Fix tests

    @yongyan-gh
    yongyan-gh committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    7c77019 View commit details
    Browse the repository at this point in the history

Commits on Feb 19, 2022

  1. Update extension methods names

    @yongyan-gh
    yongyan-gh committed Feb 19, 2022
    Configuration menu
    Copy the full SHA
    966b7f8 View commit details
    Browse the repository at this point in the history

Commits on Feb 20, 2022

  1. Change xml doc comments to normal comments

    @yongyan-gh
    yongyan-gh committed Feb 20, 2022
    Configuration menu
    Copy the full SHA
    f07f4af View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2022

  1. Merge branch 'main' into users/yongyan-gh/sortingvisitor

    @michaelcfanning
    michaelcfanning authored Mar 2, 2022
    Configuration menu
    Copy the full SHA
    f18e005 View commit details
    Browse the repository at this point in the history