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.

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

feat: do not spam comments #143

Merged
merged 4 commits into from
Sep 29, 2021
Merged

feat: do not spam comments #143

merged 4 commits into from
Sep 29, 2021

Conversation

yknx4
Copy link
Collaborator

@yknx4 yknx4 commented Sep 29, 2021

Do not create multiple comments, instead update the latest one
closes #142

@yknx4 yknx4 temporarily deployed to librecov-staging September 29, 2021 20:48 Inactive
@github-actions github-actions bot added the test label Sep 29, 2021
@librecov
Copy link

librecov bot commented Sep 29, 2021

Librecov Report

Hey @yknx4

Merging features/single-comment (d426cb3) will decrease coverage by -0.01%.
The diff coverage is 58.26%.

Impacted Files Coverage Δ
lib/opencov/services/github/auth.ex 69.57% <-4.51%> (Δ) ⬇️
lib/opencov/services/github/comments.ex 45.83% <6.94%> (Δ) ⬆️
lib/opencov/services/github/config.ex 80.0% <-20.0%> (Δ) ⬇️

Continue to review full report at Librecov.

Legend
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Librecov.

@yknxopencov
Copy link

yknxopencov bot commented Sep 29, 2021

Librecov Report

Hey @yknx4

Merging features/single-comment (5a1bf0f) will decrease coverage by -0.08%.
The diff coverage is 58.44%.

Impacted Files Coverage Δ
lib/librecov_web/live/repository_live/repository_card.ex 93.33% <-3.1%> (Δ) ⬇️
lib/opencov/services/github/auth.ex 69.57% <-4.51%> (Δ) ⬇️
lib/opencov/services/github/checks.ex 71.43% <3.43%> (Δ) ⬆️
lib/opencov/services/github/comments.ex 45.83% <6.94%> (Δ) ⬆️
lib/opencov/services/github/config.ex 100.0% <0.0%> (ø)

Continue to review full report at Librecov.

Legend
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Librecov.

@librecov
Copy link

librecov bot commented Sep 29, 2021

Librecov Report

Hey @yknx4

Merging features/single-comment (5a1bf0f) will increase coverage by 0.18%.
The diff coverage is 58.44%.

Impacted Files Coverage Δ
lib/opencov/services/github/auth.ex 69.57% <-4.51%> (Δ) ⬇️
lib/opencov/services/github/comments.ex 45.83% <6.94%> (Δ) ⬆️
lib/opencov/services/github/config.ex 100.0% <0.0%> (ø)
lib/opencov/services/github/checks.ex 71.43% <3.43%> (Δ) ⬆️

Continue to review full report at Librecov.

Legend
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Librecov.

@yknx4 yknx4 merged commit 021a827 into main Sep 29, 2021
@yknx4 yknx4 deleted the features/single-comment branch September 29, 2021 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update PR comment instead of spamming
1 participant