-
Notifications
You must be signed in to change notification settings - Fork 557
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
Pre-PR Feature Approval #87
Comments
Thanks @dendrondal. I have really fallen behind on this repo, so I'll try to make some time over the holiday to catch up. If you (or anyone else seeing this!) have an interest in helping to maintain the project, give me a shout. |
After many busy months, I finally ended up getting back to this. PR #93 covers all that's mentioned above, as well as adding Dockerization. I'd be interested in joining as a maintainer, just bear in mind that my LaTeX knowledge is VERY limited, so I'm certainly unqualified to review any PRs or answer any issues relating to it. Anything else though, feel free to tag me! |
Thanks @dendrondal
Mine too, so we can muddle through this together. I've sent you an invite! |
So my Dockerization is ready to go, I want to pull in the changes from #95, but figured I'd go ahead and submit a PR for review. I did have a question, however: how to we want to go about adding the Docker implementation to Travis? These are the options:
If you want me to submit the PR and then make the decision, that's fine too. |
thanks @dendrondal. this sounds good to me, as long as you don't see any issues. extra build time isn't a problem as far as i'm concerned, unless it leads to build failures as a result of exceeding time limits (which I don't will be the case: https://docs.travis-ci.com/user/customizing-the-build/#build-timeouts). |
Hello, I wanted to thank you for this repository. It made a huge difference in my writing process. I made several quality-of-life changes along the way that should be ready to merge into master, I just thought I'd get your opinion here first before bombarding you with PRs. Here are the proposed changes:
All this is contained in my fork if you want to look at it more closely. Please let me know which of these that you would like me to submit as PRs.
The text was updated successfully, but these errors were encountered: