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 a README listing all the layers #3454

Closed

Conversation

StreakyCobra
Copy link
Contributor

List all available layers with a one-line description for each.

Page rendered on github can be viewed here: https://github.com/StreakyCobra/spacemacs/tree/feature/layers-list/layers#layers

List all available layers with a one-line description for each.
@d12frosted
Copy link
Contributor

👍 like this idea

Also it would be nice to have those short descriptions in SPC f e h.

@StreakyCobra
Copy link
Contributor Author

It's a good idea. It would imply forcing a one-liner description of each module, in an easily parse-able/loadable place. What do you suggest? If we do this we can then use these information to generate this page.

This PR can still be used in the meantime to provide a list to users.

@robbyoconnor
Copy link
Contributor

This is awesome.

@TheBB
Copy link
Contributor

TheBB commented Oct 27, 2015

Yes, this is nice, but my kneejerk reaction is that it would fit in with a kind of layer metadata structure, so that we wouldn't have to work hard to keep this up to date (not that we merge layers that often, but still)…

@StreakyCobra
Copy link
Contributor Author

It would be better of course to setup a system that can generate this page automatically and allow to give one-liner description in helm-spacemacs. But in the meantime this could be used to have a "documentation" of layers 😄 (I wrote this just after someone asked where to look for layers on Gitter). It shouldn't be to much of a hassle to maintain, if layer's author are filling it on new layer's PR.

If you don't feel using it, don't worry, I didn't took me too much time to write, and I had already forgotten it until @d12frosted pinged it 😂

@d12frosted
Copy link
Contributor

Haha. That's me, bringing forgotten goodies 😄

Yes, this is nice, but my kneejerk reaction is that it would fit in with a kind of layer metadata structure, so that we wouldn't have to work hard to keep this up to date (not that we merge layers that often, but still)…

What other metadata could be there? Tags? 'Maintainer'? Hm...

@TheBB
Copy link
Contributor

TheBB commented Oct 30, 2015

As part of the documentation work in #3604 I wrote a small function to auto-generate a file with an overview of all layers. It doesn't have any descriptions, though. It looks like I can merge those changes this weekend. Maybe you want to do something on top of that instead?

@StreakyCobra
Copy link
Contributor Author

Yep, forgot this one, it's a lame idea to do it manually. I close it 😄

@StreakyCobra StreakyCobra deleted the feature/layers-list branch November 1, 2015 14:24
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.

4 participants