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 review comment line #16

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Add review comment line #16

merged 1 commit into from
Jan 14, 2025

Conversation

tim-thacker-nullify
Copy link
Member

the review comment webhook comment object has line numbers

image

@tim-thacker-nullify tim-thacker-nullify changed the base branch from master to main January 14, 2025 00:31
@tim-thacker-nullify tim-thacker-nullify added the minor Minor version updates (features) label Jan 14, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 12758576526

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 15 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.2%) to 88.583%
Files with Coverage Reduction New Missed Lines %
github/github.go 6 94.96%
azuredevops/azuredevops.go 9 78.33%
Totals Coverage Status
Change from base Build 10594209340: 0.2%
Covered Lines: 900
Relevant Lines: 1016
💛 - Coveralls

@tim-thacker-nullify tim-thacker-nullify merged commit 4aefbf1 into main Jan 14, 2025
12 of 13 checks passed
@tim-thacker-nullify tim-thacker-nullify deleted the add-review-comment-line branch January 14, 2025 00:34
@tim-thacker-nullify tim-thacker-nullify removed the request for review from jonathanlam January 14, 2025 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
minor Minor version updates (features)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tim-thacker-nullify @coveralls