From 373a33792446bd61d04cf67ad5b2ee3a550dee9a Mon Sep 17 00:00:00 2001 From: psteinroe Date: Thu, 4 Dec 2025 08:17:55 +0100 Subject: [PATCH] fix: lint urls --- bun.lock | 1 + .../src/categories.rs | 66 +++++++++---------- .../codegen/src/generate_new_analyser_rule.rs | 2 +- 3 files changed, 35 insertions(+), 34 deletions(-) diff --git a/bun.lock b/bun.lock index 718b63324..793040af6 100644 --- a/bun.lock +++ b/bun.lock @@ -1,5 +1,6 @@ { "lockfileVersion": 1, + "configVersion": 0, "workspaces": { "": { "name": "postgres_lsp", diff --git a/crates/pgls_diagnostics_categories/src/categories.rs b/crates/pgls_diagnostics_categories/src/categories.rs index 38517401e..ce18c4d09 100644 --- a/crates/pgls_diagnostics_categories/src/categories.rs +++ b/crates/pgls_diagnostics_categories/src/categories.rs @@ -13,39 +13,39 @@ // must be between `define_categories! {\n` and `\n ;\n`. define_categories! { - "lint/safety/addSerialColumn": "https://pg-language-server.com/latest/rules/add-serial-column", - "lint/safety/addingFieldWithDefault": "https://pg-language-server.com/latest/rules/adding-field-with-default", - "lint/safety/addingForeignKeyConstraint": "https://pg-language-server.com/latest/rules/adding-foreign-key-constraint", - "lint/safety/addingNotNullField": "https://pg-language-server.com/latest/rules/adding-not-null-field", - "lint/safety/addingPrimaryKeyConstraint": "https://pg-language-server.com/latest/rules/adding-primary-key-constraint", - "lint/safety/addingRequiredField": "https://pg-language-server.com/latest/rules/adding-required-field", - "lint/safety/banCharField": "https://pg-language-server.com/latest/rules/ban-char-field", - "lint/safety/banConcurrentIndexCreationInTransaction": "https://pg-language-server.com/latest/rules/ban-concurrent-index-creation-in-transaction", - "lint/safety/banDropColumn": "https://pg-language-server.com/latest/rules/ban-drop-column", - "lint/safety/banDropDatabase": "https://pg-language-server.com/latest/rules/ban-drop-database", - "lint/safety/banDropNotNull": "https://pg-language-server.com/latest/rules/ban-drop-not-null", - "lint/safety/banDropTable": "https://pg-language-server.com/latest/rules/ban-drop-table", - "lint/safety/banTruncateCascade": "https://pg-language-server.com/latest/rules/ban-truncate-cascade", - "lint/safety/changingColumnType": "https://pg-language-server.com/latest/rules/changing-column-type", - "lint/safety/constraintMissingNotValid": "https://pg-language-server.com/latest/rules/constraint-missing-not-valid", - "lint/safety/creatingEnum": "https://pg-language-server.com/latest/rules/creating-enum", - "lint/safety/disallowUniqueConstraint": "https://pg-language-server.com/latest/rules/disallow-unique-constraint", - "lint/safety/lockTimeoutWarning": "https://pg-language-server.com/latest/rules/lock-timeout-warning", - "lint/safety/multipleAlterTable": "https://pg-language-server.com/latest/rules/multiple-alter-table", - "lint/safety/preferBigInt": "https://pg-language-server.com/latest/rules/prefer-big-int", - "lint/safety/preferBigintOverInt": "https://pg-language-server.com/latest/rules/prefer-bigint-over-int", - "lint/safety/preferBigintOverSmallint": "https://pg-language-server.com/latest/rules/prefer-bigint-over-smallint", - "lint/safety/preferIdentity": "https://pg-language-server.com/latest/rules/prefer-identity", - "lint/safety/preferJsonb": "https://pg-language-server.com/latest/rules/prefer-jsonb", - "lint/safety/preferRobustStmts": "https://pg-language-server.com/latest/rules/prefer-robust-stmts", - "lint/safety/preferTextField": "https://pg-language-server.com/latest/rules/prefer-text-field", - "lint/safety/preferTimestamptz": "https://pg-language-server.com/latest/rules/prefer-timestamptz", - "lint/safety/renamingColumn": "https://pg-language-server.com/latest/rules/renaming-column", - "lint/safety/renamingTable": "https://pg-language-server.com/latest/rules/renaming-table", - "lint/safety/requireConcurrentIndexCreation": "https://pg-language-server.com/latest/rules/require-concurrent-index-creation", - "lint/safety/requireConcurrentIndexDeletion": "https://pg-language-server.com/latest/rules/require-concurrent-index-deletion", - "lint/safety/runningStatementWhileHoldingAccessExclusive": "https://pg-language-server.com/latest/rules/running-statement-while-holding-access-exclusive", - "lint/safety/transactionNesting": "https://pg-language-server.com/latest/rules/transaction-nesting", + "lint/safety/addSerialColumn": "https://pg-language-server.com/latest/reference/rules/add-serial-column/", + "lint/safety/addingFieldWithDefault": "https://pg-language-server.com/latest/reference/rules/adding-field-with-default/", + "lint/safety/addingForeignKeyConstraint": "https://pg-language-server.com/latest/reference/rules/adding-foreign-key-constraint/", + "lint/safety/addingNotNullField": "https://pg-language-server.com/latest/reference/rules/adding-not-null-field/", + "lint/safety/addingPrimaryKeyConstraint": "https://pg-language-server.com/latest/reference/rules/adding-primary-key-constraint/", + "lint/safety/addingRequiredField": "https://pg-language-server.com/latest/reference/rules/adding-required-field/", + "lint/safety/banCharField": "https://pg-language-server.com/latest/reference/rules/ban-char-field/", + "lint/safety/banConcurrentIndexCreationInTransaction": "https://pg-language-server.com/latest/reference/rules/ban-concurrent-index-creation-in-transaction/", + "lint/safety/banDropColumn": "https://pg-language-server.com/latest/reference/rules/ban-drop-column/", + "lint/safety/banDropDatabase": "https://pg-language-server.com/latest/reference/rules/ban-drop-database/", + "lint/safety/banDropNotNull": "https://pg-language-server.com/latest/reference/rules/ban-drop-not-null/", + "lint/safety/banDropTable": "https://pg-language-server.com/latest/reference/rules/ban-drop-table/", + "lint/safety/banTruncateCascade": "https://pg-language-server.com/latest/reference/rules/ban-truncate-cascade/", + "lint/safety/changingColumnType": "https://pg-language-server.com/latest/reference/rules/changing-column-type/", + "lint/safety/constraintMissingNotValid": "https://pg-language-server.com/latest/reference/rules/constraint-missing-not-valid/", + "lint/safety/creatingEnum": "https://pg-language-server.com/latest/reference/rules/creating-enum/", + "lint/safety/disallowUniqueConstraint": "https://pg-language-server.com/latest/reference/rules/disallow-unique-constraint/", + "lint/safety/lockTimeoutWarning": "https://pg-language-server.com/latest/reference/rules/lock-timeout-warning/", + "lint/safety/multipleAlterTable": "https://pg-language-server.com/latest/reference/rules/multiple-alter-table/", + "lint/safety/preferBigInt": "https://pg-language-server.com/latest/reference/rules/prefer-big-int/", + "lint/safety/preferBigintOverInt": "https://pg-language-server.com/latest/reference/rules/prefer-bigint-over-int/", + "lint/safety/preferBigintOverSmallint": "https://pg-language-server.com/latest/reference/rules/prefer-bigint-over-smallint/", + "lint/safety/preferIdentity": "https://pg-language-server.com/latest/reference/rules/prefer-identity/", + "lint/safety/preferJsonb": "https://pg-language-server.com/latest/reference/rules/prefer-jsonb/", + "lint/safety/preferRobustStmts": "https://pg-language-server.com/latest/reference/rules/prefer-robust-stmts/", + "lint/safety/preferTextField": "https://pg-language-server.com/latest/reference/rules/prefer-text-field/", + "lint/safety/preferTimestamptz": "https://pg-language-server.com/latest/reference/rules/prefer-timestamptz/", + "lint/safety/renamingColumn": "https://pg-language-server.com/latest/reference/rules/renaming-column/", + "lint/safety/renamingTable": "https://pg-language-server.com/latest/reference/rules/renaming-table/", + "lint/safety/requireConcurrentIndexCreation": "https://pg-language-server.com/latest/reference/rules/require-concurrent-index-creation/", + "lint/safety/requireConcurrentIndexDeletion": "https://pg-language-server.com/latest/reference/rules/require-concurrent-index-deletion/", + "lint/safety/runningStatementWhileHoldingAccessExclusive": "https://pg-language-server.com/latest/reference/rules/running-statement-while-holding-access-exclusive/", + "lint/safety/transactionNesting": "https://pg-language-server.com/latest/reference/rules/transaction-nesting/", // end lint rules // splinter rules start "splinter/performance/authRlsInitplan": "https://supabase.com/docs/guides/database/database-advisors?lint=0003_auth_rls_initplan", diff --git a/xtask/codegen/src/generate_new_analyser_rule.rs b/xtask/codegen/src/generate_new_analyser_rule.rs index c085bf03f..d65a1cbf8 100644 --- a/xtask/codegen/src/generate_new_analyser_rule.rs +++ b/xtask/codegen/src/generate_new_analyser_rule.rs @@ -129,7 +129,7 @@ pub fn generate_new_analyser_rule( // We sort rules to reduce conflicts between contributions made in parallel. let rule_line = match category { Category::Lint => format!( - r#" "lint/{group}/{rule_name_camel}": "{PGLS_WEBSITE}/latest/rules/{kebab_case_rule}","# + r#" "lint/{group}/{rule_name_camel}": "{PGLS_WEBSITE}/latest/reference/rules/{kebab_case_rule}/","# ), }; let lint_start = match category {