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

Regularly check for kVariant updates #192

Merged
merged 2 commits into from
Mar 1, 2023
Merged

Conversation

goodhoko
Copy link
Contributor

@goodhoko goodhoko commented Feb 15, 2023

Related issue: #185

Add a workflow that checks if there are any updates to the kVariants dictionary mirrored from https://github.com/hfhchan/irg and in such case submit a PR.

Schedule it to run at 10:00 every Monday but also allow it to be triggered manually.

An example of a run can be seen here and the resulting (testing) PR over here (note: the diff set is empty because the artificial change I introduced for testing has since been reverted).

Closes #185.

Add a workflow that checks if there are any updates to the kVariants
dictionary mirrored from https://github.com/hfhchan/irg and in such case
submit a PR.

Schedule it to run at 10:00 every Monday but also allow it to be
triggered manually.

Closes meilisearch#185.
@ManyTheFish
Copy link
Member

Hello @goodhoko, so we have to add a CI env var named MEILI_BOT_GH_PAT, that has the writing access on this repository, am I right?

@goodhoko
Copy link
Contributor Author

goodhoko commented Feb 23, 2023

@ManyTheFish Sorry I should've mention this in the PR description. I took the variable from the other workflow that alters the repo so I assumed it could (and should) be used in this workflow as well.

Though, it's not strictly necessary to use a custom access token. If you enable the "Allow GitHub Actions to create and approve pull requests" option at the very bottom of the Actions settings the GH_TOKEN can be just left set to the default value.

ManyTheFish
ManyTheFish previously approved these changes Feb 27, 2023
Copy link
Member

@ManyTheFish ManyTheFish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I Approve this PR,
I request @curquiza to check the CI.

@@ -0,0 +1,29 @@
name: Check for kVariants updates
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does not look like a CI check to me, but more a CI that updates the dictionaries.
I would also add a comment to describe the CI, like "Create a PR update the dictionaries if a changes has been found" (for example)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@curquiza Just pushed a fixup commit that should address both of your remarks. 🙌

Copy link
Member

@ManyTheFish ManyTheFish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @goodhoko for your work, and thank you @curquiza for the double check!

bors merge

@bors
Copy link
Contributor

bors bot commented Mar 1, 2023

Build succeeded:

@bors bors bot merged commit 33c9196 into meilisearch:main Mar 1, 2023
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.

Create a CI keeping kvariants up-to-date
3 participants