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

Make Usage section a bit more straight-forward. #278

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

whatacold
Copy link

Hi,

Thanks for this repo.

I'm a newbie of LaTeX, it takes me a few minutes to figure out that xelatex {your-cv}.tex in README means to compile it with cv.tex or resume.tex in examples/ dir.
At first I think it's ok to run with every single .tex files there, which of course leads to compilation errors.

So I think it's better to take the exact file as an example in README.

README.md Outdated
@@ -83,14 +83,14 @@ You can [install TeX from upstream](http://tex.stackexchange.com/q/1092) (recomm

#### Usage

At a command prompt, run
At a command prompt, run (note that ``$`` is shell prompt)

Choose a reason for hiding this comment

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

Not needed.

README.md Outdated

```bash
$ xelatex {your-cv}.tex
$ cd examples/
$ xelatex cv.tex

Choose a reason for hiding this comment

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

Just adding cd examples could be useful.

$ cd examples
$ xelatex {your-cv}.tex

```

This should result in the creation of ``{your-cv}.pdf``

Choose a reason for hiding this comment

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

Not necessary.

@whatacold
Copy link
Author

Sorry about the late commit.

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.

None yet

2 participants