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

Report in JSON Format Still Includes Markdown Link #115

Closed
aj-stein-nist opened this issue May 10, 2022 · 5 comments
Closed

Report in JSON Format Still Includes Markdown Link #115

aj-stein-nist opened this issue May 10, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@aj-stein-nist
Copy link

Hello, already a big fan of this simple, elegant tool you wrote! I am working on using it to check our website content for a project. I noticed that, even when I pick format JSON, the file is inevitably a JSON file end with a Markdown link.

This is the result of the run from that action. I noticed prior to using GHA but here is an example.

https://github.com/aj-stein-nist/OSCAL/actions/runs/2301231596

html_link_report.zip

Is this intentional, can you/I/we consider generating the JSON without the additional non-JSON footer?

@aj-stein-nist
Copy link
Author

Sigh, I put this in the wrong project, it was meant for the lychee-action repo. Would you like to move this or prefer I make another one? Apologies!

@mre mre transferred this issue from lycheeverse/lychee May 10, 2022
@mre
Copy link
Member

mre commented May 10, 2022

Wow, apparently it checked 1400 links/second. Nice. 🤩
Moved the issue to lychee-action, no worries.

We have a check for that, but apparently it doesn't work:

if [ "${INPUT_FORMAT}" == "markdown" ]; then
echo "[Full Github Actions output](${GITHUB_WORKFLOW_URL})" >> "${INPUT_OUTPUT}"

Need to investigate a bit, but it's certainly a bug. Thanks for reporting.

@mre mre added the bug Something isn't working label May 10, 2022
@aj-stein-nist
Copy link
Author

Awesome and timely, thanks @mre. Maybe this week I can try to fix the bug and contribute back. Sharing is caring afterall.

Thanks for the tool, big improvement over the alternatives. Looks like it is time to brush off my tabs to learn Rust and contribute back more than easier bash entrypoint fixes. 😅

@mre
Copy link
Member

mre commented May 10, 2022

Actually can you try uses: lycheeverse/lychee-action@master? I think the latest version is just not released yet. If that works I'll release v1.4.2 with the fix.

@mre
Copy link
Member

mre commented May 30, 2022

Version 1.5.0 with the fix was just released. ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants