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

Additional info for readme. #146

Merged
merged 3 commits into from
Nov 14, 2019

Conversation

markgdawson
Copy link
Contributor

I sure anyone using stumpwm would work this out pretty quickly, but I don't see any harm in
having the information ready there.

I sure anyone using stumpwm would work this out pretty quickly, but I don't see any harm in
having the information ready there.
@@ -21,6 +21,12 @@ In order to get your own notifications handler you can define on function

** Dependencies

To install dependencies (if you are using quicklisp):
Copy link
Member

Choose a reason for hiding this comment

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

A shorter way to install the dependencies is to load the system itself, ie. (ql:quickload "notify"). btw one can pass in a list of system names to quickload

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the pointers! 👍

@PuercoPop
Copy link
Member

Thanks for improving the setup instrutions, I left a minor comment, other than that it looks good. 👍

@markgdawson
Copy link
Contributor Author

Based on your comment, I added some info to the top level README - is this a good idea do you think?

Probably obvious to many people, but I didn't realise this! Maybe because I'm not too familiar with quicklisp yet.

@PuercoPop PuercoPop merged commit d981d18 into stumpwm:master Nov 14, 2019
@PuercoPop
Copy link
Member

This good idea do you think?

Yes, my impression is that most StumpWM users have little prior CL experience so providing clear instructions and not assuming prior knowledge of the CL ecosystem is the best way to go regarding documentation.

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.

2 participants