liquidtux icon indicating copy to clipboard operation
liquidtux copied to clipboard

Match file locations with (expected) kernel ones.

Open amezin opened this issue 3 years ago • 7 comments

  • Move sources to drivers/hwmon.

  • Move docs to Documentation/hwmon.

Closes #33

Signed-off-by: Aleksandr Mezin [email protected]

amezin avatar Jan 19 '23 06:01 amezin

While you're at it, you can also delete nzxt-smartdevice.rst. As far as I can see/remember, it's just a mistake that 00bcb937be27caf0ee0a6fb0f6c191474c858e0b didn't remove it.

jonasmalacofilho avatar Jan 19 '23 11:01 jonasmalacofilho

Please rebase to remove duplicate commits (now that #41 has been merged).

jonasmalacofilho avatar Jan 21 '23 11:01 jonasmalacofilho

@jonasmalacofilho Please take a look at Settings -> Branches -> Branch protection rules -> Require status checks to pass before merging -> Require branches to be up to date before merging

Also, is anything wrong with "Rebase and merge" method? It keeps the original commit message

amezin avatar Jan 21 '23 13:01 amezin

@jonasmalacofilho So, may I merge this?

While you're at it, you can also delete nzxt-smartdevice.rst

Should be a separate change/pull request IMO

amezin avatar Feb 21 '23 10:02 amezin

Nothing wrong here, but let's not move files from under #44.

That can be resolved by git rebase automatically (but not GitHub UI though). I've already checked by rebasing this PR after https://github.com/liquidctl/liquidtux/commit/e5b8c7165496cf21329d0ff290adf4e5e5e19c2e. File moved in one branch, modified in place in another -> automatic conflict resolution.

Meanwhile this pull request not being merged blocks me from further progress - I have to work with master branch, then manually fix this pull request (when the changes affect Makefile, for example).

cc @aleksamagicka

amezin avatar Feb 22 '23 09:02 amezin

Meanwhile this pull request not being merged blocks me from further progress - I have to work with master branch, then manually fix this pull request (when the changes affect Makefile, for example).

Your argument is that you're annoyed to have to do this, so I should push the burden on the other person?

If that's it, then sorry, no, an open – and nearly ready – PR with driver changes takes priority.

jonasmalacofilho avatar Feb 22 '23 10:02 jonasmalacofilho

anyway, this PR should be updated

stalkerg avatar Mar 28 '23 09:03 stalkerg