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
feat: add neofetch
spec
#1416
feat: add neofetch
spec
#1416
Conversation
Thanks to @Alonely0 for the spec! |
Overviewsrc/neofetch.ts:Info: |
Hello @grant0417,
Please add a 👍 as a reaction to this comment to show that you read this. |
src/neofetch.ts
Outdated
}, | ||
{ | ||
name: "terminal", | ||
description: "That hAcKeR black window you type on", |
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.
lol
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 was on a cafe with my tablet doing the spec (my laptop's screen broke, and that's the only place with decent WiFi around) and I had like five 7-8 year olds around me calling me hacker because I was writing a JSON. They gave me the idea.
src/neofetch.ts
Outdated
{ | ||
name: "--speed_type", | ||
description: "Change the type of cpu speed to display", | ||
args: [ |
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.
Pretty sure this should be suggestions
src/neofetch.ts
Outdated
{ | ||
name: "--cpu_cores", | ||
description: "Whether or not to display the number of CPU cores", | ||
args: [{ name: "logical" }, { name: "physical" }, { name: "off" }], |
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.
Same
src/neofetch.ts
Outdated
{ | ||
name: "--cpu_temp", | ||
description: "Hide/Show cpu temperature", | ||
args: [{ name: "C" }, { name: "F" }, { name: "off" }], |
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.
Same
src/neofetch.ts
Outdated
{ | ||
name: "--gpu_type", | ||
description: "Which GPU to display", | ||
args: [{ name: "all" }, { name: "dedicated" }, { name: "integrated" }], |
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.
Same
src/neofetch.ts
Outdated
{ | ||
name: "--disk_subtitle", | ||
description: "What information to append to the Disk subtitle", | ||
args: [ |
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.
Same
src/neofetch.ts
Outdated
{ | ||
name: "--memory_unit", | ||
description: "Memory output unit", | ||
args: [{ name: "kib" }, { name: "mib" }, { name: "gib" }], |
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.
You get the gist haha
Thanks to you for fixing all the typos I made on the tablet & debugging it! |
@grant0417 can you give me write access so I can fix my mess? |
Not sure i can give you perms |
Then test the gist on my other comment and if it works well, commit it. There should be a box somewhere saying something in the lines of "allow maintainers/reviewers to commit/edit" though. |
Okay, put your gist in |
Everything's green, @separaterecords if the spec works approve the PR |
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.
Everything seems to be ok, but I can't test the spec because as I said earlier my laptop's screen is broken and I'm doing everything on my tablet. Shouldn't be merged until someone tests it.
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 looks good to me! Thanks @Alonely0!
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Additional info: