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: fix activity types of the event watch #334

Merged
merged 1 commit into from
Aug 5, 2023
Merged

fix: fix activity types of the event watch #334

merged 1 commit into from
Aug 5, 2023

Conversation

suzuki-shunsuke
Copy link
Contributor

@suzuki-shunsuke suzuki-shunsuke commented Aug 5, 2023

The activity type `starred` is wrong.
The correct activity type is `started`.

- github/docs#27127
- github/docs#27129
@suzuki-shunsuke suzuki-shunsuke marked this pull request as ready for review August 5, 2023 00:50
@rhysd
Copy link
Owner

rhysd commented Aug 5, 2023

@suzuki-shunsuke Thank you for catching it and fixing the upstream. The upstream change was automatically imported at #335 along with other changes in action inputs. But I'll merge this PR for crediting you.

@rhysd rhysd merged commit c29244f into rhysd:main Aug 5, 2023
12 checks passed
@suzuki-shunsuke
Copy link
Contributor Author

Oh, I'm sorry that I missed the document. 🙇‍♂️

https://github.com/rhysd/actionlint/blob/main/CONTRIBUTING.md#maintain-all_webhooksgo

Thank you for merging this pull request.

@suzuki-shunsuke suzuki-shunsuke deleted the fix-watch-activity-types branch August 5, 2023 12:12
@suzuki-shunsuke
Copy link
Contributor Author

@rhysd I'm sorry to bother you, but do you have any plan to release a new version?
I really appreciate if you release a new version for this fix. 🙏

@suzuki-shunsuke
Copy link
Contributor Author

I created an issue. #351

@rhysd
Copy link
Owner

rhysd commented Sep 13, 2023

@suzuki-shunsuke Would you check #351? I'll try to do it by this weekend. Sorry for the delay.

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