Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@chrisduerr
Copy link
Contributor

No description provided.

Copy link
Member

@Drakulix Drakulix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but CI doesn't seem to be happy with your doc-comment example.

@codecov-commenter
Copy link

Codecov Report

Patch coverage has no change and project coverage change: -0.07 ⚠️

Comparison is base (f7851c1) 23.58% compared to head (7630873) 23.52%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1004      +/-   ##
==========================================
- Coverage   23.58%   23.52%   -0.07%     
==========================================
  Files         132      134       +2     
  Lines       21113    21171      +58     
==========================================
  Hits         4980     4980              
- Misses      16133    16191      +58     
Flag Coverage Δ
wlcs-core 20.65% <0.00%> (-0.06%) ⬇️
wlcs-output 8.73% <0.00%> (-0.03%) ⬇️
wlcs-pointer-input 22.78% <0.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/wayland/idle_inhibit/inhibitor.rs 0.00% <0.00%> (ø)
src/wayland/idle_inhibit/mod.rs 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@Drakulix Drakulix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for protocol implementation :)

@Drakulix Drakulix merged commit e13f4c9 into Smithay:master May 5, 2023
@chrisduerr chrisduerr deleted the idle_inhibit branch May 5, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants