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

Readability improvements to documentation #184

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benagricola
Copy link
Contributor

Simplifies a number of sentences, cleans up redundant wording, fixes a number of spelling errors and introduces further consistency when mentioning constants such as HIT and MISS.

Simplifies a number of sentences, cleans up redundant wording,
fixes a number of spelling errors and introduces further
consistency when mentioning constants such as HIT and MISS.

Signed-off-by: Ben Agricola <bagricola@squiz.co.uk>
@pintsized
Copy link
Member

Also see PR #178 - @hamishforbes did we drop luarocks support for some reason? I guess either we need a new rockspec or the docs need to be updated for OPM as per that PR?

@hamishforbes
Copy link
Collaborator

🤷‍♂️We never actually used OPM or rocks for installation internally, it was always deployed straight from git for ... reasons.

I think we wanted to move to OPM but the lack of support for external (luarocks) dependencies was a problem?

In short, I don't know :)

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