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

Fixed syntax error in update-action-to-use-docker-image-to-run-action #64

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

SeanTAllen
Copy link
Member

It turns out our CI linting would show green for code that had failed linting. That is a bad setup. In the process, the last release had a syntax error in it.

This commit fixes that issue. The pylint problem still needs to be addressed.

It turns out our CI linting would show green for code that had failed linting.
That is a bad setup. In the process, the last release had a syntax error in it.

This commit fixes that issue. The pylint problem still needs to be addressed.
@SeanTAllen SeanTAllen added the changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge label Aug 29, 2023
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Aug 29, 2023
@SeanTAllen SeanTAllen merged commit 11db3c0 into main Aug 29, 2023
10 checks passed
@SeanTAllen SeanTAllen deleted the fix-broken-release branch August 29, 2023 22:39
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Aug 29, 2023
github-actions bot pushed a commit that referenced this pull request Aug 29, 2023
github-actions bot pushed a commit that referenced this pull request Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants