After saveReady module - Githubissues

processwire-recipes / Recipes

Recipe Markdown files

29 stars 18 forks source link

After saveReady module #14

Closed LostKobrakai closed 9 years ago

LostKobrakai commented 9 years ago

Sorry for all those commits, used github to preview it :smile:

owzim commented 9 years ago

Hey, thanks for the contribution! :+1:

Couple of things I'd like to mention:

We use plural lingo on tags when applicable, so it would be modules instead of module
I am unsure about the title, can we make it shorter? also: "do things", hmmm ... :)
We use 4 space indenting for all the other recipes, it'd be cool if you could change that.
Perhaps also backtick the path in the text?

Looking forward to the revision.

owzim commented 9 years ago

@LostKobrakai If you're on Mac you can preview the MarkDown with Mou.app unfortunately it does not support language ids for code blocks, still.

Not sure about Windows, a quick Google resulted in MarkdownPad being pretty popular.

LostKobrakai commented 9 years ago

I've Marked 2 installed, just was lazy.

marcus-herrmann commented 9 years ago

And if you don't want to install a particular app for this, and you're using Sublime Text or PHPStorm (or any other JetBrains IDE), there are plugins available, for example:

owzim commented 9 years ago

The Sublime Text plugin works very spotty (no refresh an such), and the output has always been weird for me, and totally different from GitHub.

owzim commented 9 years ago

Can we somehow merge it without all the sub commits?

LostKobrakai commented 9 years ago

I don't think. At least not without me deleting my fork and adding the file newly as a whole thing.