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
mode 1.21, tests and log msg txt #144
Conversation
Summary CoverItUp: kevincobain2000/gobrew
Generated by Cover It Up. |
Summary CoverItUp: kevincobain2000/gobrew
Generated by Cover It Up. |
Summary CoverItUp: kevincobain2000/gobrew
Generated by Cover It Up. |
gobrew_test.go
Outdated
@@ -186,7 +186,8 @@ func TestDoNotUpgradeLatestVersion(t *testing.T) { | |||
|
|||
gb := NewGoBrewDirectory(tempDir) | |||
|
|||
binaryDir := filepath.Join(gb.installDir, "bin") | |||
binaryDir := filepath.Join(gb.installDir, "bin", "gobrew") |
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.
I don’t understand much, why is binaryDir changing here? This is the directory where the file will be located.
Why does the location of the binary file change?
As a result we will have:
~/.gobrew/bin/gobrew/gobrew
This reverts commit 94cd26a.
Summary CoverItUp: kevincobain2000/gobrew
Generated by Cover It Up. |
Summary CoverItUp: kevincobain2000/gobrew
Generated by Cover It Up. |
Hi @juev
2 FYI items
FYI-1) I have added branch name
json
to the Settings -> Protected Branches (force push allowed due to below)gobrew/.github/workflows/sync-release-tags.yml
Line 30 in 4aabed8
FYI-2) I have also removed some old branches that have been merged.