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

Bump numerous dependencies #230

Merged
merged 12 commits into from
Apr 3, 2023
Merged

Bump numerous dependencies #230

merged 12 commits into from
Apr 3, 2023

Conversation

polomani
Copy link
Contributor

@polomani polomani commented Mar 30, 2023

This PR is to fix CVEs primarily, but also update the codebase where required.

  • migrate to @grafana/create-plugin from @grafana/toolkit (new recommended way for plugin dev)
  • all files in .config/* are generated by @grafana/create-plugin

Those files are also generated by @grafana/create-plugin:

  • .eslintrc
  • .nvmrc
  • .prettierrc.js
  • docker-compose.yaml
  • jest-setup.js
  • jest.config.js
  • tsconfig.json

Dependencies and commands in package.json are updated by the script as well.

Actual changes:

  • added getDataSourceWithMocks so we can initiate data source for every describe(..), the previous approach doesn't seem to work with the new setup
  • moved test_utils outside of tests folder
  • updated "@grafana/data": "^9.4.7", "@grafana/runtime": "^9.4.7", "@grafana/ui": "^9.4.7"

This needs to be tested with Grafana v8 / v9

Bumped Node to v16 in GitHub actions.

@polomani polomani changed the title [WIP] Bump dependencies Bump dependencies Mar 31, 2023
@polomani polomani marked this pull request as ready for review March 31, 2023 14:39
@polomani polomani requested a review from a team March 31, 2023 14:39
build:
context: ./.config
args:
grafana_version: ${GRAFANA_VERSION:-9.3.8}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we should try this with v8.5~ and 9.X

@polomani polomani changed the title Bump dependencies Bump numerous dependencies Apr 3, 2023
@polomani polomani merged commit fb5fd08 into master Apr 3, 2023
@polomani polomani deleted the bump-dependencies branch April 3, 2023 13:00
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.

2 participants