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

Improve doc label experience #2997

Closed
tisonkun opened this issue Dec 25, 2023 · 5 comments · Fixed by #2998
Closed

Improve doc label experience #2997

tisonkun opened this issue Dec 25, 2023 · 5 comments · Fixed by #2998

Comments

@tisonkun
Copy link
Collaborator

tisonkun commented Dec 25, 2023

What type of enhancement is this?

Other

What does the enhancement do?

In Pulsar, we use https://github.com/apache/pulsar-test-infra/tree/master/docbot to auto label PRs with doc needed or no needed.

Currently, we have a workflow for doing this but it will always fail before any triager properly label it.

Implementation challenges

Perhaps we depend on https://github.com/apache/pulsar-test-infra/tree/master/docbot; or we create a new repo to host greptime intra-repos actions.

@tisonkun
Copy link
Collaborator Author

cc @nicecui @sunng87 @waynexia

@waynexia
Copy link
Member

Thanks for your information, do you know if Pulsar has any plan to provide it on GitHub Action Market? If I read it correctly, we have to depend on those go code directly to make it work

@waynexia
Copy link
Member

I find a similar extension https://github.com/marketplace/actions/regex-issue-labeler, let me have a try

@tisonkun
Copy link
Collaborator Author

tisonkun commented Dec 25, 2023

if Pulsar has any plan to provide it on GitHub Action Market

I don't think so. We use pulsar-test-infra for hosting intra-repos actions. But perhaps I can make one personally.

@waynexia
Copy link
Member

Submits #2998

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 a pull request may close this issue.

2 participants