Skip to content

Update README.rst#165

Closed
NullYing wants to merge 1 commit intoJazzCore:masterfrom
NullYing:patch-1
Closed

Update README.rst#165
NullYing wants to merge 1 commit intoJazzCore:masterfrom
NullYing:patch-1

Conversation

@NullYing
Copy link
Copy Markdown

Error: caskroom/cask was moved. Tap homebrew/cask-cask instead.

Error: caskroom/cask was moved. Tap homebrew/cask-cask instead.
Copy link
Copy Markdown

@BastianZim BastianZim left a comment

Choose a reason for hiding this comment

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

I would suggest changing the brew cask command.

Comment thread README.rst
.. code-block:: bash

$ brew install caskroom/cask/wkhtmltopdf
$ brew install homebrew/cask-cask/wkhtmltopdf
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
$ brew install homebrew/cask-cask/wkhtmltopdf
$ brew cask install wkhtmltopdf

As outlined in #149 (comment) brew specifies the correct method as brew cask install.

I have also created #168 as I hadn't seen this one but only one is necessary so I can close mine as well.

@ArtemBernatskyy
Copy link
Copy Markdown

it seems that this repo has died, xD

@BastianZim
Copy link
Copy Markdown

Yeah, unfortunately.

@JazzCore JazzCore mentioned this pull request Mar 19, 2021
@JazzCore
Copy link
Copy Markdown
Owner

brew cask install wkhtmltopdf does not work for me on current MacOS, but brew install homebrew/cask/wkhtmltopdf does, fixed in #173, thanks guys and sorry that it took so long to get to it.

@JazzCore JazzCore closed this Mar 19, 2021
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