fix: switching between zh and en bug#720
Conversation
|
|
| console/atest-desktop/coverage | ||
| atest-store-git | ||
| .db | ||
| .marscode/ No newline at end of file |
There was a problem hiding this comment.
Usually, we put a new line at the end of each file. You can search the reason in Google.
There was a problem hiding this comment.
ok,I will search it.
tools/make/common.mk
Outdated
| @echo -e "Usage:\n make \033[36m<Target>\033[0m \033[36m<Option>\033[0m\n\nTargets:" | ||
| @awk 'BEGIN {FS = ":.*##"; printf ""} /^[a-zA-Z_0-9-]+:.*?##/ { printf " \033[36m%-15s\033[0m %s\n", $$1, $$2 } /^##@/ { printf "\n\033[1m%s\033[0m\n", substr($$0, 5) } ' $(MAKEFILE_LIST) | ||
| @echo -e "\n$$USAGE_OPTIONS" | ||
|
|
There was a problem hiding this comment.
IMO, this mk target more appropriate to put it in golang .mk
There was a problem hiding this comment.
Thank you for your correction. I will read the code carefully again.
LinuxSuRen
added
bug
LinuxSuRen
changed the title
LinuxSuRen
left a comment
LinuxSuRen
left a comment
There was a problem hiding this comment.
I have tested the function. It works well. It's great. Thanks for your efforts. I learn from it.
By the way, the go fmt command is duplicated. Please let me know how to reproduce the error.
tools/make/common.mk
Outdated
| .PHONY: fmt | ||
| fmt: | ||
| go fmt ./... |
There was a problem hiding this comment.
It already exists. You can find it from golang.mk
There was a problem hiding this comment.
So, what is your pre-commit hook command?
|
4 participants
What type of PR is this?
fix:switching between zh and en bug
What this PR does / why we need it:
This PR enables real-time switching from English to Chinese without requiring a page refresh. In addition, the modification to console/atest-ui/src/views/TestSuite.vue addresses a warning that appeared during npm run dev.
Because I have configured a pre-commit hook, the following error occurred during commit:
To fix this, I modified
tools/make/common.mkto recursively format all Go files under the current directory and its subdirectories.Which issue(s) this PR fixes:
Fixes # issues/710