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

Support Hugo only? #27

Closed
hacdias opened this issue Oct 12, 2015 · 6 comments
Closed

Support Hugo only? #27

hacdias opened this issue Oct 12, 2015 · 6 comments

Comments

@hacdias
Copy link
Member

hacdias commented Oct 12, 2015

There's something else here that can be a problem: right now, the plugin can be used with more static website generators, but if it was directed only to Hugo it could be more optimized and have better functionalities both in the editor (because we know the default front matter fields) and in the back end because we know the default website structure. It's easier. It can be optimized, it can be easily better.

I know I thought before it was better to support many website generators, but there's nothing good which bring bad things too. Almost nothing. We have to see the pros and cons. Do you prefer having the plugin optimized for Hugo with better specific functionalities or do you prefer it to be general and support multiple static generators?

What do you think in restricting this editor to Hugo?

@adprof
Copy link

adprof commented Oct 12, 2015

I'm all for it. But keep code modular to leave room for contributors and their forks in the future.
There are no other generators besides Hugo ;)

@mholt
Copy link

mholt commented Oct 12, 2015

I vote to keep it focused - I'd rather have a plugin just for hugo. It'll be easier to maintain and use.

@hacdias
Copy link
Member Author

hacdias commented Oct 13, 2015

So I can do that. I think it's better too. When I've time, I will change it to "caddy-hugo" again and make the changes on csddyserver.com repo and in the registry. I'll wait for your answer @mholt!

Sure it will keeo being modular as it is @adprof!

@mholt
Copy link

mholt commented Oct 13, 2015

Sounds good to me... make the directive hugo again as well, instead of cms? Since the directive is changing, maybe wait to push it to master until a new Caddy version comes out so users aren't totally surprised. Don't forget to update the docs too. (You can push to another branch other than master in the mean time.)

@hacdias
Copy link
Member Author

hacdias commented Oct 13, 2015

@mholt yes. Why waiting? =P I think it wouldn't be a great problem.

This was referenced Oct 13, 2015
@hacdias
Copy link
Member Author

hacdias commented Oct 16, 2015

TODO: change the directive and repository name.

@hacdias hacdias closed this as completed Oct 18, 2015
hacdias added a commit that referenced this issue Jul 18, 2017
Add ctrl-S save shortcut
hacdias added a commit that referenced this issue May 21, 2019
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

No branches or pull requests

3 participants