-
Notifications
You must be signed in to change notification settings - Fork 69
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
dist: build linux/darwin arm64 targets #232
Conversation
Codecov Report
@@ Coverage Diff @@
## master #232 +/- ##
==========================================
- Coverage 21.68% 0.00% -21.69%
==========================================
Files 8 8
Lines 853 854 +1
==========================================
- Hits 185 0 -185
- Misses 625 854 +229
+ Partials 43 0 -43
Continue to review full report at Codecov.
|
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.
server/zoom/meeting.go
Outdated
@@ -4,7 +4,7 @@ | |||
package zoom | |||
|
|||
// Meeting is defined at https://marketplace.zoom.us/docs/api-reference/zoom-api/meetings/meeting | |||
type Meeting struct { | |||
type Meeting struct { // nolint: govet |
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.
What linter error does this ignore?
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.
a bit hard to organize the struct, if you want to try I appreciate
or maybe I'm too dumb that I cannot organize the struct
Summary
Adding the linux/darwin arm64 targets, we start building arm64 for the server and will start adding the targets for the plugins for the ones we prepackage in the build
related to: mattermost/mattermost#18491
Ticket Link
related to https://mattermost.atlassian.net/browse/DOPS-633