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

fix: Backslash conversion for Windows environment #223

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

FlowingSPDG
Copy link

Windows環境でvolcago -o gen -p task-c task Task を実行した際に、ModelImportPathのパス区切りが逆スラッシュになってしまった問題を修正

@FlowingSPDG
Copy link
Author

すみません、友人のPCを使用していためcommitが @execut1ve 扱いとなっておりますが無視いただけますと幸いです 🙏

Copy link
Member

@54m 54m left a comment

Choose a reason for hiding this comment

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

LGTM

修正ありがとうございました!

今回の対応外のところで linter が怒ってるので、後ほど対応しておきます。
CI 自体は通っているので、マージします。

@54m 54m merged commit 7db5e17 into go-generalize:master Feb 14, 2024
2 checks passed
@FlowingSPDG
Copy link
Author

迅速な対応ありがとうございます! 🙏

L87の.go がconstantではない警告の部分でしょうか?
image
こちらも差し支えなければ、issue/PRの作成の方行っておきます!

@FlowingSPDG FlowingSPDG deleted the issues/222/main branch February 14, 2024 11:25
@54m
Copy link
Member

54m commented Feb 14, 2024

@FlowingSPDG ですです!
お願いできますか?

プロジェクトルートで、 make lint を実行すれば他にもおかしいところがあれば検出できるのですが、

  • 現在ローカルに golangci-lint が入っている前提で動作するようになっていること
  • .github/.golangci.yml の情報が古い

などありますので、こちらは別でやっておきます!

@FlowingSPDG
Copy link
Author

@54m issueの方作成しました! #227

make lintの件承知です、ありがとうございます!

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.

Windows環境の場合にModelImportPathが逆スラッシュになる
3 participants