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

Fixed lint warnings in Grafana raised by Mixtool #22486

Merged
merged 2 commits into from
Jan 17, 2023

Conversation

Dasomeone
Copy link
Contributor

What this PR does / why we need it:

This PR introduces a few minor changes to the gitea-monitoring-mixin, specifically linting issues raised by Mixtool:

  • Query selectors using job and instance have been update to allow multi-select
  • Added missing attributes to job and instance template

As this change is very minor I haven't created an issue, but please let me know if you'd like me to do so. According to the guidelines, it seemed to only be for larger designs :)

@zeripath zeripath changed the title Fixed lint warnings raised by Mixtool Fixed lint warnings in Grafana raised by Mixtool Jan 17, 2023
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 17, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 17, 2023
@delvh delvh added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Jan 17, 2023
@jolheiser jolheiser merged commit 7a0f2fb into go-gitea:main Jan 17, 2023
@jolheiser jolheiser added skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. and removed reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. labels Jan 17, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 18, 2023
* upstream/main:
  Simplify the error message when `index.js` couldn't be loaded (go-gitea#22354)
  Support asciicast files as new markup (go-gitea#22448)
  Support scoped access tokens (go-gitea#20908)
  some refactor about code comments (go-gitea#20821)
  docs: add swagger.json file location to FAQ (go-gitea#22489)
  docs: bump Gitea version (go-gitea#22490)
  chore: changelog 1.18.1 (go-gitea#22471) (go-gitea#22487)
  Fixed lint warnings in Grafana raised by Mixtool (go-gitea#22486)
  Set disable_gravatar/enable_federated_avatar when offline mode is true (go-gitea#22479)
  Fix pull request API field `closed_at` always being `null` (go-gitea#22482)
  Fix migration from gitbucket (repost) (go-gitea#22477)
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants