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

feat: add actionlint #89

Merged

Conversation

adomani
Copy link
Contributor

@adomani adomani commented Aug 2, 2024

I find this very useful. I simply copied the relevant file from mathlib.

@bryangingechen bryangingechen merged commit 2654ea3 into leanprover-community:master Aug 2, 2024
2 checks passed
@bryangingechen
Copy link
Contributor

Thanks! It's certainly saved me before when working on mathlib workflows.

bryangingechen added a commit that referenced this pull request Aug 2, 2024
@bryangingechen
Copy link
Contributor

Ah, it seems the blog is not compatible with Python 3.12 yet: https://github.com/leanprover-community/blog/actions/runs/10218978646/job/28276284146

@adomani
Copy link
Contributor Author

adomani commented Aug 2, 2024

Actually, I do not need python 3.12 and I think that actionlint does not require it. I'll try with an earlier version.

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