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

Switch the examples to use action v4 #119

Merged
merged 1 commit into from
Feb 3, 2024
Merged

Switch the examples to use action v4 #119

merged 1 commit into from
Feb 3, 2024

Conversation

bernhardkaindl
Copy link
Contributor

@bernhardkaindl bernhardkaindl commented Jan 30, 2024

Updated: Converted to draft because v4 is not ready yet, I started testing and it fails.
Do not merge!

After action-mypy v4 is tested and an improvement over v3, and you want new users to use v4 instead of v3 you may update the examples in README.md to use tsuyoshicho/action-mypy@v4.

Of course, this is for just after you are confident in v4 to be ready.

PS, and off-topic: This PR includes that change too: While at it, you may also want to update actions/checkout from v2 to v4 because, as far as I am aware, there should be no reason for new users to copy the use of the use of older v2 of actions/checkout

Signed-off-by: Bernhard Kaindl <bernhardkaindl7@gmail.com>
@bernhardkaindl bernhardkaindl marked this pull request as draft January 30, 2024 04:51
@tsuyoshicho tsuyoshicho self-requested a review January 30, 2024 12:41
@tsuyoshicho
Copy link
Owner

Hi @bernhardkaindl
Thank you for sending me the detailed corrections.

@tsuyoshicho
Copy link
Owner

This is not a problem and will be merged

@tsuyoshicho tsuyoshicho marked this pull request as ready for review February 3, 2024 00:17
@tsuyoshicho tsuyoshicho merged commit e1b6f62 into tsuyoshicho:master Feb 3, 2024
8 checks passed
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