-
Notifications
You must be signed in to change notification settings - Fork 7.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
ci(dockerfile): Fixed entrypoint.sh is not executable issue (IDFGH-12681) #13672
Conversation
👋 Hello xiahualiu, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Good point, thank you for the contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This was merged by 248d25a. (It will appear in a couple of days) Thank you for the contribution. |
This is the new pull request for the already closed PR #13660 due to previous commit issues.
The new commit was created with the
pre-commit
hooks.Besides the original change, there are a few new changes which were introduced by the
pre-commit
hooks, including word spelling, etc.Attached pre-commit log:
Thank you for your time.