Page MenuHomeSolus

Update nheko to add runtime dependency
AbandonedPublic

Authored by nazar on Aug 5 2023, 9:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 27, 1:48 AM
Unknown Object (File)
Sat, Aug 24, 4:06 AM
Unknown Object (File)
Fri, Aug 23, 10:06 PM
Unknown Object (File)
Fri, Aug 23, 9:20 PM
Unknown Object (File)
Fri, Aug 16, 7:06 PM
Unknown Object (File)
Sat, Aug 10, 1:16 PM
Unknown Object (File)
Aug 8 2024, 10:50 PM
Unknown Object (File)
Aug 8 2024, 3:57 AM
Subscribers

Details

Summary

adding rundep bc. on budgie nheko would display blank
screen unless qt5-graphicaleffects was manually installed

resolves T10614

Test Plan

join solus channels; say hello;

Diff Detail

Repository
R5651 nheko
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 5375
Build 5375: arc lint + arc unit

Event Timeline

nazar requested review of this revision.Aug 5 2023, 9:03 PM
TClark77 requested changes to this revision.Aug 5 2023, 9:11 PM
TClark77 added a subscriber: TClark77.

Thanks for contributing this patch. Please update the task summary and the commit message to be more in line with our standards
https://github.com/solus-project/tooling-central/blob/master/README.rst#using-git

  • The task name should be "Update nheko to add runtime dependency"
  • Update the commit message to explain exactly what's being done and the reasoning (per the link above)
  • Add "resolves T10614" to the commit message.

The patch itself looks good.

This revision now requires changes to proceed.Aug 5 2023, 9:11 PM

Update nheko to add runtime dependency

nazar retitled this revision from resolves T10614 to Update nheko to add runtime dependency.Aug 5 2023, 9:26 PM
nazar edited the summary of this revision. (Show Details)
TClark77 requested changes to this revision.Aug 5 2023, 9:32 PM

Thanks for updating the task name and adding the link. My apologies, I missed one other thing.

The summary of the commit should be the same as the summary of this task
Please amend your commit with this. I'll let Reilly approve the commit since he's the maintainer.

This revision now requires changes to proceed.Aug 5 2023, 9:32 PM

Hi Nazar! My apologies but I saw the initial bug report but not that there was an open diff for it. I committed the fix and only then noticed this.