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

[Feature] Add Chinese Comment Check Workflow to Kratos CI #3422

Open
mo3et opened this issue Sep 13, 2024 · 10 comments · May be fixed by #3426
Open

[Feature] Add Chinese Comment Check Workflow to Kratos CI #3422

mo3et opened this issue Sep 13, 2024 · 10 comments · May be fixed by #3426
Labels

Comments

@mo3et
Copy link
Contributor

mo3et commented Sep 13, 2024

Please see the FAQ in our main README.md before submitting your issue.

Requirements description of the feature

I wrote a workflow for our team to check for Chinese comments in the repository when a Pull Request (PR) is opened. The workflow is currently used in our team repo, and you can find it here.

I would like to this workflows is useful to Kratos's workflows. If so, I will submit a PR to integrate this into the repo.

References

workflows details

workflows effect

@mo3et mo3et added the feature label Sep 13, 2024
@mo3et
Copy link
Contributor Author

mo3et commented Sep 13, 2024

@shenqidebaozi @Windfarer

@kratos-ci-bot
Copy link
Collaborator

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


@shenqidebaozi @Windfarer

@shenqidebaozi
Copy link
Member

This situation should not exist in Kratos. I think you can first check if there are any problems in the current warehouse

@mo3et
Copy link
Contributor Author

mo3et commented Sep 14, 2024

There are no issues with the current repo. This workflow is triggered when a contributor creates a PR and checks the submitted code, similar to how Go Test ensures that the submitted content can run properly when a PR is initiated.

The check only runs when a PR is opened. If the contributor's code contains Chinese comments, the CI will report an error.

@mo3et
Copy link
Contributor Author

mo3et commented Sep 14, 2024

If you think this issue is not useful, please close it.

@shenqidebaozi
Copy link
Member

You can submit a PR, add an action, and add a Chinese comment. I would like to see the actual effect

@mo3et
Copy link
Contributor Author

mo3et commented Sep 18, 2024

You can submit a PR, add an action, and add a Chinese comment. I would like to see the actual effect

Fine. I will commit to it later today.

@mo3et mo3et linked a pull request Sep 18, 2024 that will close this issue
Copy link

dosubot bot commented Dec 18, 2024

Hi, @mo3et. I'm Dosu, and I'm helping the Kratos team manage their backlog. I'm marking this issue as stale.

Issue Summary

  • You suggested adding a workflow to detect Chinese comments in code during PRs.
  • @shenqidebaozi questioned its necessity but later encouraged you to submit a PR.
  • You agreed to proceed with a PR to demonstrate the workflow's functionality.
  • The team is open to testing the feature before deciding on integration.

Next Steps

  • Please let me know if this issue is still relevant to the latest version of the Kratos repository by commenting here.
  • If there is no further activity, this issue will be automatically closed in 7 days.

Thank you for your understanding and contribution!

@dosubot dosubot bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Dec 18, 2024
@mo3et
Copy link
Contributor Author

mo3et commented Dec 19, 2024

Hi, @mo3et. I'm Dosu, and I'm helping the Kratos team manage their backlog. I'm marking this issue as stale.

Issue Summary

  • You suggested adding a workflow to detect Chinese comments in code during PRs.
  • @shenqidebaozi questioned its necessity but later encouraged you to submit a PR.
  • You agreed to proceed with a PR to demonstrate the workflow's functionality.
  • The team is open to testing the feature before deciding on integration.

Next Steps

  • Please let me know if this issue is still relevant to the latest version of the Kratos repository by commenting here.
  • If there is no further activity, this issue will be automatically closed in 7 days.

Thank you for your understanding and contribution!

@shenqidebaozi this PR need to solve.

@dosubot dosubot bot removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Dec 19, 2024
Copy link

dosubot bot commented Dec 19, 2024

@shenqidebaozi, the user @mo3et has indicated that this issue is still relevant and needs your assistance to move forward with the PR for detecting Chinese comments in code. Could you please help them with this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants