pypandoc icon indicating copy to clipboard operation
pypandoc copied to clipboard

Collect pandoc urls from github api

Open nstrmx opened this issue 1 year ago • 7 comments

nstrmx avatar Aug 23 '24 06:08 nstrmx

Hey @nstrmx thanks for the PR.

Can you take a look at the failing checks. Do you think it's a PR issue, or something else?

JessicaTegner avatar Aug 23 '24 08:08 JessicaTegner

Fixed it (hopefully).

nstrmx avatar Aug 26 '24 07:08 nstrmx

Interestingly @nstrmx it seems that the single test failing, is actually because of a 403 rate limit. What would you say we could do about that? Asking because Idon't know that much about GHs api, when used in this way.

JessicaTegner avatar Sep 17 '24 05:09 JessicaTegner

If we add support for the GITHUB_TOKEN as an environment variable to be used in requests, will that solve this?

nstrmx avatar Oct 20 '24 15:10 nstrmx

yes, that would be acceptable. But I feel like that if that key isn't set, we should fall back to the old behaviour. Due to the nature of pypandoc, where it's used a lot in CI/CD workflows, I do think that if it's not set, we should fall back to old behaviour so stuff doesn't break

JessicaTegner avatar Oct 20 '24 16:10 JessicaTegner

Should work now.

nstrmx avatar Oct 26 '24 18:10 nstrmx

Great work. Lastly, since we now should have 2 download behaviors, we should probably make a test casefor each behavior

JessicaTegner avatar Oct 27 '24 13:10 JessicaTegner