search

bazelbuild / bazel-central-registry

The central registry of Bazel modules for the Bzlmod external dependency system.
https://registry.bazel.build
Apache License 2.0
244 stars 303 forks source link

rules_appimage@1.12.0 #2265

Closed publish-to-bcr-bot[bot] closed 3 months ago

publish-to-bcr-bot[bot] commented 3 months ago

Release: https://github.com/lalten/rules_appimage/releases/tag/v1.12.0

Automated by Publish to BCR

bazel-io commented 3 months ago

Hello @lalten, modules you maintain (rules_appimage) have been updated in this PR. Please review the changes.

fmeum commented 3 months ago

@lalten The failure looks significant

lalten commented 3 months ago

@lalten The failure looks significant

Yep, I was too quick to approve. The GitHub Actions CI in rules_appimage has https://github.com/lalten/rules_appimage/blob/1641c92f0b70ac141806ad013109383d0da9f2f6/.github/workflows/ci.yaml#L42, need to add this to the BCR presubmit as well

lalten commented 3 months ago

Hmm

(15:33:16) ERROR: Skipping '//... -//tests/pycross/...': invalid package name '... -//tests/pycross': package names may not contain '//' path separators
  | (15:33:16) ERROR: Failed to parse target pattern even though it was previously parsed successfully: invalid package name '... -//tests/pycross': package names may not contain '//' path separators

How can pass this target pattern to the presubmit CI? 🤔