Skip to content

[MM-176] Sync the plugin with starter template#451

Merged
mickmister merged 3 commits intomasterfrom
MM-176
Feb 1, 2024
Merged

[MM-176] Sync the plugin with starter template#451
mickmister merged 3 commits intomasterfrom
MM-176

Conversation

@ayusht2810
Copy link
Copy Markdown
Contributor

lieut-data and others added 3 commits January 31, 2024 15:59
#192)

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

This reverts commit be4a281d0cc791d10e6e5ae917b325b2f054e475.

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

This reverts commit ca9ee3c17c6920a636a33f378e17395afd6f329f.

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

This reverts commit 18d30b50bc7ba800c9f05bfd82970781db0aea3e.

* 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>
Signed-off-by: ayusht2810 <ayush.thakur@brightscout.com>
Co-authored-by: Jesse Hallam <jesse.hallam@gmail.com>
@ayusht2810 ayusht2810 self-assigned this Jan 31, 2024
@ayusht2810 ayusht2810 added the 2: Dev Review Requires review by a core committer label Jan 31, 2024
Comment on lines +184 to +191
### Releasing new versions

The version of a plugin is determined at compile time, automatically populating a `version` field in the [plugin manifest](plugin.json):
* If the current commit matches a tag, the version will match after stripping any leading `v`, e.g. `1.3.1`.
* Otherwise, the version will combine the nearest tag with `git rev-parse --short HEAD`, e.g. `1.3.1+d06e53e1`.
* If there is no version tag, an empty version will be combined with the short hash, e.g. `0.0.0+76081421`.

To disable this behaviour, manually populate and maintain the `version` field.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Interesting. So you can no longer inspect plugin.json to see what version a plugin is at

@mickmister mickmister merged commit 7834d0f into master Feb 1, 2024
@mickmister mickmister deleted the MM-176 branch February 1, 2024 05:16
@mickmister mickmister mentioned this pull request Feb 16, 2024
@ayusht2810 ayusht2810 mentioned this pull request Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2: Dev Review Requires review by a core committer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants