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

Chore: upgrade grafana deps #122

Merged
merged 5 commits into from
Nov 11, 2022
Merged

Chore: upgrade grafana deps #122

merged 5 commits into from
Nov 11, 2022

Conversation

ryantxu
Copy link
Member

@ryantxu ryantxu commented Nov 10, 2022

This should have no real runtime effects, but is bumping most of the build dependencies.

@@ -395,7 +395,8 @@ export class QueryEditor extends PureComponent<Props, State> {
if (!this.props.datasource.grafanaLiveEnabled) {
return null;
}

// Fixing weird typing issue
const FVM = FieldValidationMessage as any;
Copy link
Member Author

Choose a reason for hiding this comment

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

This one is weird 😬 --in 9x it is ok, but the 8.3x part has weird inherited types that do not expect childe attributes.

@@ -1,6 +1,6 @@
{
"name": "grafana-iot-twinmaker-app",
"version": "1.2.1",
"version": "1.3.0-pre",
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the reason for bumping the version now?

Copy link
Member Author

Choose a reason for hiding this comment

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

🤔 -- really depends how we want to use main, but given that main is the next version, not the last version, i think it is better to bump sooner than later (it is weird to have dev artifacts with the same version as the previous release!)

@hwandersman hwandersman merged commit 22467ba into main Nov 11, 2022
@mukeshsahay mukeshsahay deleted the update-grafana-deps branch January 24, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants