Skip to content

Fix workflow permissions and requirements build.#165

Merged
copybara-service[bot] merged 1 commit intomainfrom
test_783088227
Jul 15, 2025
Merged

Fix workflow permissions and requirements build.#165
copybara-service[bot] merged 1 commit intomainfrom
test_783088227

Conversation

@copybara-service
Copy link

Fix workflow permissions and requirements build.

@copybara-service copybara-service bot force-pushed the test_783088227 branch 6 times, most recently from c256192 to 0fcda66 Compare July 15, 2025 00:48
Comment on lines +10 to +12
permissions:
contents: read
id-token: write
Copy link
Contributor

Choose a reason for hiding this comment

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

Right, it should be here.

Comment on lines 27 to 28
from array_record.python import array_record_data_source
from array_record.python import array_record_module
Copy link
Contributor

Choose a reason for hiding this comment

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

This test and the one above is executed by Bazel, rather than pytest after installing the package, so I think these paths are messed up a bit. It would be better to have them in pytest side, but for Bazel-managed tests I think we need to keep from python ....

@copybara-service copybara-service bot force-pushed the test_783088227 branch 3 times, most recently from 70fc24b to c3ede3d Compare July 15, 2025 14:39
@copybara-service copybara-service bot merged commit 5b2afea into main Jul 15, 2025
@copybara-service copybara-service bot deleted the test_783088227 branch July 15, 2025 14:40
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.

1 participant