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
Add Asus TUF FX504GD #183
Add Asus TUF FX504GD #183
Conversation
I would squash the commits |
How do you do that ? Kind of new to git and I didn't fully understand how can I squash multiple commits on a PR, thanks. |
to squash the last 3 commits into the 4th, you can do:
This will make your changes appear as 1 commit, instead of 4 |
Thanks a lot, better now ? |
yea, a lot cleaner, thanks :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI should be fine again.
@@ -128,6 +129,7 @@ See code for all available configurations. | |||
| [Tuxedo InfinityBook v4][] | `<nixos-hardware/tuxedo/infinitybook/v4>` | | |||
|
|||
[Acer Aspire 4810T]: acer/aspire/4810t | |||
[Asus TUF FX504GD]: asus/fx504gd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not yet documented but can you also add it here? https://github.com/NixOS/nixos-hardware/blob/master/flake.nix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it good ? I kind of think I messed up a little bit by trying to commit and squash, tell me if I need to change something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can squash on merge.
@@ -0,0 +1,66 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks indeed weird. I have a look later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, send a message here if I need to do something.
merged in #185 |
Request to add Asus TUF FX504GD to nixos-hardware channel.
Nouveau doesn't work at all on this laptop, so I add nouveau.modeset=0 as a kernel param.