Update URLs (#1504)
## Buildtools PR checklist
- [x] The code in this PR is covered by unit/integration tests.
- [ ] I have tested these changes and provide testing instructions
below.
- [x] I have either responded to, or resolved all Gemini comments on the
PR.
- [x] I have read Google Eng Practices on [Small
Changes](https://google.github.io/eng-practices/review/developer/small-cls.html),
this PR either follows these guidelines or the description provides
reasoning for why they can not be followed.
## Description
This follows the org move so that users don't need to be redirected all
the time.
diff --git a/build/BUILD.bazel b/build/BUILD.bazel
index e1d3f6f..b1585de 100644
--- a/build/BUILD.bazel
+++ b/build/BUILD.bazel
@@ -44,7 +44,7 @@
],
data = glob(["testdata/*"]) + [
# parse.y.go is checked in to satisfy the Go community
- # https://github.com/bazelbuild/buildtools/issues/14
+ # https://github.com/bazel-contrib/buildtools/issues/14
# this test ensures it doesn't get stale.
"parse.y.baz.go",
"parse.y.go",
diff --git a/buildifier/README.md b/buildifier/README.md
index 82ff565..a3d81fd 100644
--- a/buildifier/README.md
+++ b/buildifier/README.md
@@ -154,7 +154,7 @@
sha256 = "ae34c344514e08c23e90da0e2d6cb700fcd28e80c02e23e4d5715dddcb42f7b3",
strip_prefix = "buildtools-4.2.2",
urls = [
- "https://github.com/bazelbuild/buildtools/archive/refs/tags/4.2.2.tar.gz",
+ "https://github.com/bazel-contrib/buildtools/archive/refs/tags/4.2.2.tar.gz",
],
)
```
@@ -202,7 +202,7 @@
"actionable": true,
"autoFixable": true,
"message": "The \"/\" operator for integer division is deprecated in favor of \"//\".",
- "url": "https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#integer-division"
+ "url": "https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#integer-division"
}
]
},
diff --git a/buildifier/integration_test.sh b/buildifier/integration_test.sh
index d6da2aa..d4cc289 100755
--- a/buildifier/integration_test.sh
+++ b/buildifier/integration_test.sh
@@ -453,11 +453,11 @@
fixed test_dir/to_fix_tmp.bzl
EOF
-error_bzl="test_dir/to_fix_tmp.bzl:1: bzl-visibility: Module \"//foo/bar/internal/baz:module.bzl\" can only be loaded from files located inside \"//foo/bar\", not from \"//to_fix_tmp.bzl\". (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#bzl-visibility)"
-error_docstring="test_dir/to_fix_tmp.bzl:1: module-docstring: The file has no module docstring."$'\n'"A module docstring is a string literal (not a comment) which should be the first statement of a file (it may follow comment lines). (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#module-docstring)"
-error_integer="test_dir/to_fix_tmp.bzl:4: integer-division: The \"/\" operator for integer division is deprecated in favor of \"//\". (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#integer-division)"
-error_dict="test_dir/to_fix_tmp.bzl:5: unsorted-dict-items: Dictionary items are out of their lexicographical order. (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#unsorted-dict-items)"
-error_cfg="test_dir/to_fix_tmp.bzl:6: attr-cfg: cfg = \"data\" for attr definitions has no effect and should be removed. (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#attr-cfg)"
+error_bzl="test_dir/to_fix_tmp.bzl:1: bzl-visibility: Module \"//foo/bar/internal/baz:module.bzl\" can only be loaded from files located inside \"//foo/bar\", not from \"//to_fix_tmp.bzl\". (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#bzl-visibility)"
+error_docstring="test_dir/to_fix_tmp.bzl:1: module-docstring: The file has no module docstring."$'\n'"A module docstring is a string literal (not a comment) which should be the first statement of a file (it may follow comment lines). (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#module-docstring)"
+error_integer="test_dir/to_fix_tmp.bzl:4: integer-division: The \"/\" operator for integer division is deprecated in favor of \"//\". (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#integer-division)"
+error_dict="test_dir/to_fix_tmp.bzl:5: unsorted-dict-items: Dictionary items are out of their lexicographical order. (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#unsorted-dict-items)"
+error_cfg="test_dir/to_fix_tmp.bzl:6: attr-cfg: cfg = \"data\" for attr definitions has no effect and should be removed. (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#attr-cfg)"
test_lint () {
ret=0
@@ -554,7 +554,7 @@
"actionable": true,
"autoFixable": false,
"message": "Module \"//foo/bar/internal/baz:module.bzl\" can only be loaded from files located inside \"//foo/bar\", not from \"//json/to_fix.bzl\".",
- "url": "https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#bzl-visibility"
+ "url": "https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#bzl-visibility"
},
{
"start": {
@@ -569,7 +569,7 @@
"actionable": true,
"autoFixable": true,
"message": "The \"/\" operator for integer division is deprecated in favor of \"//\".",
- "url": "https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#integer-division"
+ "url": "https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#integer-division"
},
{
"start": {
@@ -584,7 +584,7 @@
"actionable": true,
"autoFixable": true,
"message": "cfg = \"data\" for attr definitions has no effect and should be removed.",
- "url": "https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#attr-cfg"
+ "url": "https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#attr-cfg"
}
]
},
@@ -707,7 +707,7 @@
EOF
cat > report_golden <<EOF
-BUILD:1: deprecated-function: The function "bar" defined in "//lib.bzl" is deprecated. (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#deprecated-function)
+BUILD:1: deprecated-function: The function "bar" defined in "//lib.bzl" is deprecated. (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#deprecated-function)
EOF
$buildifier --lint=warn --warnings=deprecated-function BUILD 2> report || ret=$?
@@ -737,8 +737,8 @@
EOF
cat > report_golden <<EOF
-BUILD:1: allowed-symbol-load-locations: Symbol "s1" must be loaded from :z.bzl. (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#allowed-symbol-load-locations)
-BUILD:2: allowed-symbol-load-locations: Symbol "s3" must be loaded from one of the allowed locations: :x.bzl, :y.bzl. (https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#allowed-symbol-load-locations)
+BUILD:1: allowed-symbol-load-locations: Symbol "s1" must be loaded from :z.bzl. (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#allowed-symbol-load-locations)
+BUILD:2: allowed-symbol-load-locations: Symbol "s3" must be loaded from one of the allowed locations: :x.bzl, :y.bzl. (https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#allowed-symbol-load-locations)
EOF
$buildifier --lint=warn --warnings=allowed-symbol-load-locations -tables=buildifier.tables BUILD 2> report || true
diff --git a/buildifier/npm/package.json b/buildifier/npm/package.json
index cafd38d..8daf5bd 100644
--- a/buildifier/npm/package.json
+++ b/buildifier/npm/package.json
@@ -11,9 +11,9 @@
],
"repository": {
"type": "git",
- "url": "https://github.com/bazelbuild/buildtools.git"
+ "url": "https://github.com/bazel-contrib/buildtools.git"
},
"bugs": {
- "url": "https://github.com/bazelbuild/buildtools/issues"
+ "url": "https://github.com/bazel-contrib/buildtools/issues"
}
}
diff --git a/buildozer/npm/index.js b/buildozer/npm/index.js
index fb9b337..907139d 100644
--- a/buildozer/npm/index.js
+++ b/buildozer/npm/index.js
@@ -7,7 +7,7 @@
/**
* run buildozer with a list of commands
- * @see https://github.com/bazelbuild/buildtools/tree/main/buildozer#do-multiple-changes-at-once
+ * @see https://github.com/bazel-contrib/buildtools/tree/main/buildozer#do-multiple-changes-at-once
* @returns The standard out of the buildozer command, split by lines
*/
function run(...commands) {
@@ -21,7 +21,7 @@
* @param flags any buildozer flags to pass
*/
function runWithOptions(commands, options, flags = []) {
- // From https://github.com/bazelbuild/buildtools/tree/main/buildozer#usage:
+ // From https://github.com/bazel-contrib/buildtools/tree/main/buildozer#usage:
// Here, label-list is a space-separated list of Bazel labels,
// for example //path/to/pkg1:rule1 //path/to/pkg2:rule2.
// Buildozer reads commands from FILE (- for stdin
@@ -34,7 +34,7 @@
input,
encoding: 'utf-8',
});
- // https://github.com/bazelbuild/buildtools/tree/main/buildozer#error-code
+ // https://github.com/bazel-contrib/buildtools/tree/main/buildozer#error-code
if (status == 0 || status == 3) return stdout.trim().split('\n').filter(l => !!l);
console.error(`buildozer exited with status ${status}\n${stderr}`);
diff --git a/buildozer/npm/package.json b/buildozer/npm/package.json
index bac3c22..7b26447 100644
--- a/buildozer/npm/package.json
+++ b/buildozer/npm/package.json
@@ -11,9 +11,9 @@
],
"repository": {
"type": "git",
- "url": "https://github.com/bazelbuild/buildtools.git"
+ "url": "https://github.com/bazel-contrib/buildtools.git"
},
"bugs": {
- "url": "https://github.com/bazelbuild/buildtools/issues"
+ "url": "https://github.com/bazel-contrib/buildtools/issues"
}
}
diff --git a/launcher.js b/launcher.js
index 58c2c03..04786d4 100644
--- a/launcher.js
+++ b/launcher.js
@@ -42,7 +42,7 @@
if (arch == undefined || platform == undefined) {
console.error(`FATAL: Your platform/architecture combination ${
os.platform()} - ${os.arch()} is not yet supported.
- See instructions at https://github.com/bazelbuild/buildtools/blob/main/_TOOL_/README.md.`);
+ See instructions at https://github.com/bazel-contrib/buildtools/blob/main/_TOOL_/README.md.`);
return Promise.resolve(1);
}
diff --git a/warn/warn.go b/warn/warn.go
index 7c9dd70..f86f4e4 100644
--- a/warn/warn.go
+++ b/warn/warn.go
@@ -78,7 +78,7 @@
}
func docURL(cat string) string {
- return "https://github.com/bazelbuild/buildtools/blob/main/WARNINGS.md#" + cat
+ return "https://github.com/bazel-contrib/buildtools/blob/main/WARNINGS.md#" + cat
}
// makeFinding creates a Finding object
