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

Fix running actionlint unit tests #281

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

rhysd
Copy link
Contributor

@rhysd rhysd commented Jun 20, 2023

Related: rhysd/actionlint#307.

actionlint's unit tests assume .git directory at root of repository. However, .git directory is not included in source tarballs provided by GitHub. This PR fixes the issue by making a fake .git directory.

After merging this branch, #274 should pass CI.

@rhysd rhysd changed the title actionlint: Fix running unit tests Fix running actionlint unit tests Jun 20, 2023
@aminvakil aminvakil merged commit 95aedf5 into aminvakil:master Jun 20, 2023
@aminvakil
Copy link
Owner

Thank you for fixing this!

I'll make a PR to bump actionlint to 1.6.25 now.

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