Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Support startColumn field in the SARIF report#454

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mmvpm merged 5 commits intomainfromideaseeker/add-start-column-in-sarif
Jul 8, 2022

Conversation

@mmvpm
Copy link
Collaborator

@mmvpmmmvpm commentedJul 7, 2022
edited
Loading

Description

Add a new fieldstartColumn to the SARIF report.

An empty space at the beginning of the line is not highlighted:

image

Fixes#452

Type of Change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Automated Testing

org.utbot.sarif.SarifReportTest

Manual Scenario

  1. Run test generation in any way
  2. Look at the created SARIF report
  3. Make sure it containsphysicalLocation.region.startColumn field
  4. Check the highlighted region (examples in the issueClarify the scope of error highlighting in the SARIF report #452)

Checklist:

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • The change contains enough commentaries, particularly in hard-to-understand areas
  • New documentation is provided or existed one is altered
  • No new warnings
  • Tests that prove my change is effective
  • All tests pass locally with my changes

@mmvpmmmvpm self-assigned thisJul 7, 2022
@mmvpmmmvpm requested a review fromnikitavlaevJuly 8, 2022 07:32
nikitavlaev
nikitavlaev previously requested changesJul 8, 2022
@mmvpmmmvpmforce-pushed theideaseeker/add-start-column-in-sarif branch 2 times, most recently fromf092642 to5e699f2CompareJuly 8, 2022 10:40
@mmvpmmmvpmforce-pushed theideaseeker/add-start-column-in-sarif branch from5e699f2 to1741b70CompareJuly 8, 2022 12:44
@mmvpmmmvpm requested a review fromnikitavlaevJuly 8, 2022 12:46
@mmvpmmmvpmenabled auto-merge (squash)July 8, 2022 12:56
@mmvpmmmvpm requested review fromsava-cska and removed request fornikitavlaevJuly 8, 2022 13:03
@mmvpmmmvpm merged commitf47e064 intomainJul 8, 2022
@mmvpmmmvpm deleted the ideaseeker/add-start-column-in-sarif branchJuly 8, 2022 14:14
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@sergeypospelovsergeypospelovsergeypospelov approved these changes

@nikitavlaevnikitavlaevnikitavlaev left review comments

+1 more reviewer

@sava-cskasava-cskasava-cska approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@mmvpmmmvpm

Labels

None yet

Projects

Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Clarify the scope of error highlighting in the SARIF report

5 participants

@mmvpm@sergeypospelov@nikitavlaev@sava-cska

[8]ページ先頭

©2009-2025 Movatter.jp