Skip to content

Update main.go#154

Merged
hanzei merged 1 commit intomattermost:masterfrom
chetanyakan:patch-1
Jun 1, 2021
Merged

Update main.go#154
hanzei merged 1 commit intomattermost:masterfrom
chetanyakan:patch-1

Conversation

@chetanyakan
Copy link
Copy Markdown
Contributor

Summary

  • Remove unused function: pluginIDGoFileTemplate as manifest is not generated anymore.

@mattermod
Copy link
Copy Markdown
Contributor

Hello @chetanyakan,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Copy link
Copy Markdown
Contributor

@hanzei hanzei left a comment

Choose a reason for hiding this comment

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

Good catch 👍

@hanzei hanzei requested a review from larkox May 28, 2021 05:23
@hanzei hanzei added the 2: Dev Review Requires review by a core committer label May 28, 2021
@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Jun 1, 2021
@hanzei hanzei merged commit be4a281 into mattermost:master Jun 1, 2021
@chetanyakan chetanyakan deleted the patch-1 branch June 7, 2021 08:13
lieut-data added a commit that referenced this pull request Dec 12, 2023
lieut-data added a commit that referenced this pull request Jan 23, 2024
#192)

* Revert "Update main.go (#154)"

This reverts commit be4a281.

* Revert "[MM-33506] Use embed package to include plugin manifest (#145)"

This reverts commit ca9ee3c.

* Revert "Don't generate manifest.ts (#127)"

This reverts commit 18d30b5.

* install-go-tools target, adopt gotestsum

* bring back make apply + automatic versioning

* Update build/manifest/main.go

Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>

* suppress git describe error when no tags match

* make version/release notes opt-in

* fix whitespace in Makefile

* document version management options

---------

Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>
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.

4 participants