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

docs: fix the broken link #5454

Merged
merged 2 commits into from
Oct 28, 2023

Conversation

dulltz
Copy link
Contributor

@dulltz dulltz commented Oct 26, 2023

Description

Fix the broken link in the markdown. This PR is just a trivial document issue.

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@simar7
Copy link
Member

simar7 commented Oct 26, 2023

@DmitriyLewen @knqyf263 the error seems unrelated to this change

@DmitriyLewen
Copy link
Contributor

DmitriyLewen commented Oct 27, 2023

@nikpivkin got similar error with downloading Go dependencies (go mod tidy) - https://github.com/aquasecurity/trivy/actions/runs/6611239867/job/17956478809?pr=5382
Looks like it is bug in GitHub Actions, but we didn't find way to fix this.
After few hours Action ran correct.

UPD:
I found that actions/setup-python@v4 installed CPython (3.12.0) instead of CPython (3.11.6).

@knqyf263
Copy link
Collaborator

@DmitriyLewen Do you think the python version is relevant?

@DmitriyLewen
Copy link
Contributor

Python 3.12 is latest version.

I was able to reproduce this error in Ubuntu container and am working on solution to this problem.

@DmitriyLewen
Copy link
Contributor

Created #5462 for docs issue.

@knqyf263 knqyf263 added this pull request to the merge queue Oct 28, 2023
Merged via the queue into aquasecurity:main with commit 048150d Oct 28, 2023
7 checks passed
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.

4 participants