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

Release v1.3.0 #28

Merged
merged 1 commit into from
Jul 23, 2020
Merged

Release v1.3.0 #28

merged 1 commit into from
Jul 23, 2020

Conversation

hanzei
Copy link
Contributor

@hanzei hanzei commented Jun 18, 2020

Summary

Cut release for Marketplace

Ticket Link

Fixes #27

@hanzei hanzei added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Jun 18, 2020
@hanzei hanzei added this to the v1.3.0 milestone Jun 18, 2020
@hanzei hanzei requested review from levb and iomodo June 18, 2020 08:41
@levb levb removed the 2: Dev Review Requires review by a core committer label Jun 20, 2020
@levb levb requested a review from DHaussermann June 20, 2020 04:11
@hanzei
Copy link
Contributor Author

hanzei commented Jun 20, 2020

@DHaussermann Take your time to review this plugin. There is no need to rush it. This release should go into the Marketplace.

@DHaussermann
Copy link

@hanzei I have smoke tested this plugin The functionality seems to be in order and auto-complete functionality has been added. No issues found.

However there do seem to be a couple issues with the build.

  1. When I run make I get
env GO111MODULE=off /usr/local/go/bin/go get github.com/jteeuwen/go-bindata/...
/Users/dylanhaussermann/go/bin/go-bindata -o server/memelibrary/assets.go -pkg memelibrary -prefix server/memelibrary/assets/ -ignore '(^|/)\..*' server/memelibrary/assets/...
  1. When I run make all I get
@@ -1894,4 +1895,3 @@
 	cannonicalName := strings.Replace(name, "\", "/", -1)
 	return filepath.Join(append([]string{dir}, strings.Split(cannonicalName, "/")...)...)
 }
-
Gofmt failure
make: *** [gofmt] Error 1

Would it be appropriate to create follow-up issues for this rather than stopping the release? I'm not sure of the significance of this but, I'm assuming there is no impact on the compiled binary.

@DHaussermann
Copy link

Gentle ping on this @hanzei I know you were away last week. Must these build issues ☝️ be fixed before the release or, can this move forward to the market place?

@hanzei
Copy link
Contributor Author

hanzei commented Jul 22, 2020

@DHaussermann Both build issues are not critical for the release.

  1. Good catch! I've submitted Use make all as default target #29 to fix this.
  2. with the fix from Use make all as default target #29 this issue should be encountered so often. I've filled Output from make memelibrary is not gofmt'ed #30 as a HW.

I think we are ready to ship, right?

@DHaussermann
Copy link

@hanzei yes, correct. Ready to ship.

@DHaussermann DHaussermann added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Jul 22, 2020
@hanzei hanzei merged commit 16315cd into master Jul 23, 2020
@hanzei hanzei deleted the release_v1.3.0 branch July 23, 2020 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release v1.3.0
4 participants