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

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

Merged
merged 3 commits into from
Feb 1, 2024
Merged

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

merged 3 commits into from
Feb 1, 2024

Conversation

ayusht2810
Copy link
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
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
7 checks passed
@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