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

Update URL Checker workflow #2035

Merged
merged 2 commits into from
Sep 2, 2024
Merged

Conversation

arkid15r
Copy link
Contributor

This Pull Request relates to issue #1990
This PR does not address the deprecated action update.

Update URL Checker workflow

  • merge 2 URL checker workflows (push, pr triggers)
  • limit runs to 5.0/** paths
  • bump actions/checkout from v3 to v4

  - merge 2 URL checker workflows (push, pr triggers)
  - limit runs to 5.0/** paths
  - bump actions/checkout from v3 to v4
.github/workflows/url-checker.yml Outdated Show resolved Hide resolved
.github/workflows/url-checker.yml Show resolved Hide resolved
.github/workflows/url-checker.yml Show resolved Hide resolved
Copy link
Contributor Author

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tghosth thanks for reviewing:

.github/workflows/url-checker.yml Show resolved Hide resolved
.github/workflows/url-checker.yml Outdated Show resolved Hide resolved
.github/workflows/url-checker.yml Show resolved Hide resolved
@tghosth
Copy link
Collaborator

tghosth commented Sep 2, 2024

Hi @arkid15r, thanks for your responses but I don't see that you actually made the changes to the branch in the PR?

@arkid15r
Copy link
Contributor Author

arkid15r commented Sep 2, 2024

Hi @arkid15r, thanks for your responses but I don't see that you actually made the changes to the branch in the PR?

Sorry for confusion. I forgot to update the PR yesterday.

@tghosth tghosth merged commit 2735881 into OWASP:master Sep 2, 2024
5 checks passed
@arkid15r arkid15r deleted the ark/update-url-checker branch September 2, 2024 17:29
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