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

Move testing tex file for draft and delete unused PDFs #85

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

cthoyt
Copy link
Contributor

@cthoyt cthoyt commented Sep 16, 2024

This PR fixes an issue caused by #82 and #84 being merged in quick succession, where the draft paper.tex was not put in the right folder

This PR also removes both the draft and publication PDFs from the testing resources, since #82 switched testing from PDF to tex files

@tarleb tarleb merged commit c430b5b into openjournals:main Sep 17, 2024
1 check failed
@tarleb
Copy link
Collaborator

tarleb commented Sep 17, 2024

Thanks!!

@cthoyt
Copy link
Contributor Author

cthoyt commented Sep 17, 2024

@tarleb I'm still working on making all of the tests pass. currently figuring out why the draft flag isn't getting considered correctly everywhere

@cthoyt cthoyt deleted the test-refactor-2 branch September 17, 2024 07:28
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.

2 participants