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
Toolkit: Remove deprecated plugin:build
#67485
Conversation
build:plugin
plugin:build
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.
added a question, but otherwise looks 😍😍😍
"prepack": "mv ./src ./src_bak && cp -r ./dist/src ./src", | ||
"postpack": "rimraf ./src && mv ./src_bak ./src", | ||
"postpack": "rm -rf ./src && mv ./src_bak ./src", |
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.
does this have any implications for windows machines? believe that's the whole reason for including rimraf
. or do we not care since it's now removed? 🤔
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.
The scripts are already using mv
(see prepack) which doesn't work on windows and we support windows under WSL https://grafana.com/blog/2021/03/03/how-to-set-up-a-grafana-development-environment-on-a-windows-pc-using-wsl/
Co-authored-by: Jack Westbrook <jack.westbrook@gmail.com>
What is this feature?
Removes the deprecated
plugin:build
command from toolkit.Why do we need this feature?
We are moving all plugin-related commands from toolkit to create-plugin.
Who is this feature for?
All plugin developers
Which issue(s) does this PR fix?:
Closes: #57750
Related to: #55997
Fixes #
Special notes for your reviewer:
Please check that:
Release notice breaking change
The deprecated
plugin:build
command in the Grafana Toolkit have been removed in this release. The replacementcreate-plugin
tool is recommended for plugin development.