Skip to content
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 Fig as an installation method to the README #459

Merged
merged 2 commits into from
Jun 16, 2022
Merged

Conversation

ibayramli
Copy link
Contributor

The Fig Plugin Store supports 1-click install for 400+ shell plugins. We have over 100k users, thousands of whom manage their shell configuration with Fig.

atuin is already listed in the store so we'd love to have it listed as a download method on your README.

Thanks so much and please let me know if you have any questions!

Copy link
Member

@ellie ellie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Thank you for adding support for Atuin, but I'd appreciate it if you don't try and use our README as an advertisement

README.md Outdated
@@ -192,6 +192,14 @@ And then follow [the shell setup](#shell-plugin)
Once the binary is installed, the shell plugin requires installing. If you use
the install script, this should all be done for you!

### Fig
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you move Fig down below the shell-specific installs please? We've tried to keep things roughly in the order of what people installing Atuin use most, and Fig is still super new

README.md Outdated
@@ -192,6 +192,14 @@ And then follow [the shell setup](#shell-plugin)
Once the binary is installed, the shell plugin requires installing. If you use
the install script, this should all be done for you!

### Fig

[Fig](https://fig.io) adds apps, shortcuts, and autocomplete to your existing terminal.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The description here reads like an ad - I appreciate you listing your install, but our README is not the place to sell a product or boost SEO

If you could follow the pattern of our other install methods, that would be great - purely informative, letting people who already use a given shell know how to install the plugin for it

@ibayramli
Copy link
Contributor Author

ibayramli commented Jun 16, 2022

Hi @ellie, thanks for reviewing my PR.

Could you move Fig down below the shell-specific installs please? We've tried to keep things roughly in the order of what people installing Atuin use most, and Fig is still super new

Ordering by tool popularity makes sense - I addressed it in the latest commit. My reasoning for putting Fig at top was that it's a shell-agnostic installation method that works for zsh, bash, or fish - so it made sense to me to list more general install methods above less general ones (just as you put the script-installation at the top, for example).

The description here reads like an ad - I appreciate you listing your install, but our README is not the place to sell a product or boost SEO.

Sorry to hear that the description felt like an ad - I removed it in the final commit. We just wanted to give a short description of what Fig is since we realize some users are not familiar with it.

How do my changes look to you @ellie?

Copy link
Collaborator

@conradludgate conradludgate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me now. Thanks for the update

@ellie ellie merged commit 6718db1 into atuinsh:main Jun 16, 2022
@ellie
Copy link
Member

ellie commented Jun 16, 2022

Yup this is great, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants