Skip to content

Conversation

@travagliad
Copy link

This PR upgrades and fixes dependencies, resolves lint errors, and enforces linting via git hooks.

Packages updated

  • All possible packages were updated to their latest compatible versions except those listed below.

Packages not updated (npm outdated)

  • @playwright/test and playwright: Remained at 1.44.1 (latest is 1.54.1). Updating these would upgrade the browser, causing failures with Grafana (root cause unknown). To avoid breaking tests, these were not updated.
  • mongodb: Remained at 3.7.2 (latest is 6.17.0). Updating this caused all MongoDB connection tests to fail due to breaking changes. This will be addressed in a separate ticket.
  • @types/node, @typescript-eslint/eslint-plugin, @typescript-eslint/parser, typescript-eslint: Minor updates available, but not critical for current functionality.

Security

  • npm audit reports 0 vulnerabilities.

Other activities

  • All lint errors were fixed.
  • Git hooks were added to enforce linting on commit.

travagliad and others added 4 commits July 15, 2025 11:19
* PMM-7: Fix anotations and pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix pgsql dashboards

* PMM-7: Fix mongo upgrade

* PMM-7: Fix external upgrade

* PMM-7: Fix external upgrade

* PMM-7: Fix qan

* PMM-7: Fix qan

* PMM-7: Fix qan

* PMM-7: Fix qan

* PMM-7: Fix qan

* PMM-7: Fix qan

* PMM-7: PR comment fixes

* PMM-7: PR comment fixes
@travagliad
Copy link
Author

@travagliad travagliad marked this pull request as ready for review July 17, 2025 19:16
@travagliad travagliad self-assigned this Jul 17, 2025
@travagliad
Copy link
Author

alertRulesPage.openMoreMenu(ruleName);
I.click(alertRulesPage.buttons.deleteAlertRule);
I.waitForText(alertRulesPage.messages.confirmDelete, iaCommon.elements.modalDialog);
I.waitForText(alertRulesPage.messages.confirmDelete, undefined, iaCommon.elements.modalDialog);
Copy link
Contributor

Choose a reason for hiding this comment

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

undefined?

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.

4 participants