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

Notes about installation via homebrew-cask #2

Merged
merged 1 commit into from
Oct 14, 2022
Merged

Notes about installation via homebrew-cask #2

merged 1 commit into from
Oct 14, 2022

Conversation

oschrenk
Copy link

@oschrenk oschrenk commented Jun 4, 2015

No description provided.

@@ -30,6 +38,7 @@ A couple of default styles lifted from [Marked.app][marked].

[mmdql]: https://github.com/fletcher/MMD-QuickLook
[screenshot]: https://github.com/ttscoff/MMD-QuickLook/raw/master/MMDQLGithub-1.jpg
[cask]: http://caskroom.io/

Choose a reason for hiding this comment

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

This should be homebrew-cask, not simply cask.

Copy link
Author

Choose a reason for hiding this comment

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

Why? The link works in Marked2 app and also on https://github.com/oschrenk/MMD-QuickLook/tree/homebrew-cask-readme

According to http://daringfireball.net/projects/markdown/syntax#link the id is the the second bracket pair

This is [an example] [id] reference-style link.

Then, anywhere in the document, you define your link label like this, on a line by itself:

[id]: http://example.com/  "Optional Title Here"

Choose a reason for hiding this comment

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

Ah, yes, you are correct. I was thinking cask would be the name to appear, but homebrew-cask is. My mistake.

@ttscoff ttscoff merged commit 723a981 into ttscoff:master Oct 14, 2022
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