@Bigpod put the review in github so we can start practicing to do it
@Bigpod put the review in github so we can start practicing to do it
Jand then I'll add mine
Jand then update the readme
Bi dont have review permission
Jhuh, I set them all to have that on yesterday
Bmaybe some setting is wrong becuase it says reviewers with write access, and it also could be since i do not have any contributions anywhere in ublue-os org
Joh, good opportuntiy to test the docs, one sec!
JBahh yea been a while since i have done reviews in github or reviews at all
Jyeah, because I ran into this too
Jyou can review but you don't know it unless you start at the bottom link
JThis seems to be all you'd need to copy:
Joh, duh, of course
Jthat makes sense, just use the root
Jlol how obviously simple
Badded a approval review
JIt basically gives you
rpm -ivh -like powersJ@Bigpod I'll fix the docs as a PR, and then if you can review the 3 PRs that are left we'd be done with this bad boy
Jand then I'll post an update in the forum like how jstone did for the nvidia one
Jok, pushed, if you'd do the honors!
JI forgot to add the release-action to this repo
B
Joh nice catch!
Bapproved both yubikey and docs one
Bbut docs one could use better name
Jwhich part? (also, there's a little widget in github where you can click on the exact part and leave a comment right next to it)
Jlike if you click the + on the left hand side of the code block
Badd openrgb to docs is a bit missleading since you do add more to docs then just openrgb information, its not realy a problem since its a simple change
JI left a review
Joh, yeah, I should specify, it's only the udev rules, not "auto set up openrgb", is that what you mean?
Byea
Bwith such small changes it isnt a problem but with big changes sometimes reviewers need to rely on title
Jyeah
Jhuh I can't seem to find it?
Ball pull request in udev-rules approved from my side
Bcant find it either
JAh, I didn't submit
JShould be able to see it now
Jugh I started a new branch instead of pushing to the same one
Jok sorry for the github noise, I think we're good now
B@.jstone @j0rge is rpm right command or should it be rpm-ostree?
JYou can go either way
JOr just include another line for
rpm-ostreedidnt know that
JI used
rpm as an example because there are tons of containers using yum/dnfJBut all of these containers, bet they rpm-ostree, yum, or dnf, can install vs rpm
Bim gonna test it before i approce
Jok this gets way easier when you have write access, welcome to the team bigpod!
Bthanks
Jalright, udev rules solved for the forseeable future
KDid we include OpenRGB?
