Skip to content

fix: BasedPyright error in test_basic_docs - #698

Closed
a-zw wants to merge 3 commits into
eclipse-score:mainfrom
a-zw:fix-needs-json-version-in-template
Closed

fix: BasedPyright error in test_basic_docs#698
a-zw wants to merge 3 commits into
eclipse-score:mainfrom
a-zw:fix-needs-json-version-in-template

Conversation

@a-zw

@a-zw a-zw commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Fixes BasedPyright error: 'Object of type 'None' is not subscriptable' on result.artifacts.

Added type guard assert result.artifacts is not None before subscripting.

Also removed duplicate copyright header (bonus).

Checklist:

  • bazel mod tidy passes
  • Pre-commit hooks pass
  • BasedPyright clean

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 04668b81-06cc-461c-b073-4cf73ae08d83
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 10.864s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@a-zw a-zw closed this Aug 6, 2026
@a-zw
a-zw deleted the fix-needs-json-version-in-template branch August 6, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant