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 typo in pylint example, unmatched " Update README.rst #520

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

clintonsteiner
Copy link
Collaborator

@clintonsteiner clintonsteiner commented Oct 18, 2023

Change: --lint="pylint --ignore='setup.py'"" to --lint="pylint --ignore='setup.py'" to be correct

@clintonsteiner
Copy link
Collaborator Author

Hello @MatthijsBurgh
I appreciate you taking the time to review my PR.
Any advice on getting the github actions to kickoff?

Oddly enough I did the same thing on my fork and they ran immediately.

@clintonsteiner
Copy link
Collaborator Author

Looks like the other two PR's haven't had the actions kickoff either. Let me know if I should open up an issue
#519
#515

@MatthijsBurgh
Copy link
Collaborator

We need @akaihola for the CI

@akaihola
Copy link
Owner

We need @akaihola for the CI

I guess I didn't react in time. I can't find how to re-run the checks now. Any insight?

@akaihola akaihola added bug Something isn't working documentation Improvements or additions to documentation labels Feb 23, 2024
@MatthijsBurgh
Copy link
Collaborator

I have no clue why the CI is not being run.

@akaihola akaihola force-pushed the clintonsteiner-patch-1 branch 3 times, most recently from 8ba56ba to 16cf013 Compare February 26, 2024 22:01
@akaihola akaihola self-requested a review February 26, 2024 22:22
Change: --lint="pylint --ignore='setup.py'"" to --lint="pylint --ignore='setup.py'" to be correct
@akaihola akaihola force-pushed the clintonsteiner-patch-1 branch from 16cf013 to b933b7c Compare February 26, 2024 22:23
@akaihola akaihola merged commit 8bf0b6b into akaihola:master Feb 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
Development

Successfully merging this pull request may close these issues.

4 participants