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

Create Plugin: Remove links pointing to deprecated starter templates #162

Merged
merged 2 commits into from
Nov 29, 2022

Conversation

jackw
Copy link
Collaborator

@jackw jackw commented Nov 29, 2022

What this PR does / why we need it:
Generated links in plugin.json incorrectly point to the deprecated starter templates. This PR removes them.

Which issue(s) this PR fixes:

Fixes #129

Special notes for your reviewer:

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @grafana/create-plugin@0.7.1-canary.162.e431ddf.0
# or 
yarn add @grafana/create-plugin@0.7.1-canary.162.e431ddf.0

@jackw jackw added bug create-plugin related to the create-plugin tool patch Increment the patch version when merged release Create a release when this pr is merged labels Nov 29, 2022
@jackw jackw added this to the v1.0.0 milestone Nov 29, 2022
@jackw jackw self-assigned this Nov 29, 2022
@@ -257,7 +257,7 @@ function printSuccessMessage(answers: CliArgs) {
];

return `
Congratulations on scaffolding a Grafana ${answers.pluginName} plugin! 🚀
Congratulations on scaffolding a Grafana ${answers.pluginType} plugin! 🚀
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Spotted this mistake I made in previous PR whilst testing.

Copy link
Collaborator

@leventebalogh leventebalogh left a comment

Choose a reason for hiding this comment

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

👍

@jackw jackw merged commit 2419c37 into main Nov 29, 2022
@jackw jackw deleted the jackw/update-links-plugin-json branch November 29, 2022 11:28
@grafanabot
Copy link
Contributor

🚀 PR was released in @grafana/create-plugin@0.7.1 🚀

@grafanabot grafanabot added the released This issue/pull request has been released. label Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-plugin related to the create-plugin tool patch Increment the patch version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Generated links in plugin.json incorrectly points to the starter template
3 participants