possible fix for #54

[aghaynes]

this hopefully filters the last line of install_win.bat from the tinytex install which is stopping the install on windows at the moment (see #54)

I say hopefully as it is untested. A colleague helped with this, and feels that it could work though...

[codecov-io]

Codecov Report

Merging #103 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #103   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            1         1           
=========================================
  Hits             1         1           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ff8d11...d447928. Read the comment docs.

[jimhester]

I realize you have put a fair amount of work into this and I apologize for not looking at what you were doing earlier. However these actions are written in typescript, e.g. in https://github.com/r-lib/actions/blob/master/setup-tinytex/src/setup-tinytex.ts which is then compiled to javascript.

So rather than editing the javascript directly you need to edit the typescript source, then call npm run format && npm run build to generate the javascript files.

[aghaynes]

thanks @jimhester I made the changes (hopefully correctly, as I say, I don't have much of a clue about js). I did not commit changes that my installation of npm made to the package*.yaml files, which related to different versions of prettify (or something similar)

[aghaynes]

FWIW, I tested this on a build (by referencing my fork in the workflow file as opposed to the r-lib one), and it seems to work (no error on compiling the PDF vignette)... see https://github.com/aghaynes/secuTrialR-1/runs/684459045

[jimhester]

Thanks again for working on this!

[aghaynes]

Thanks Jim! :)

[github-actions]

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue and include a link to this pull request.