Skip to content
This repository was archived by the owner on May 7, 2026. It is now read-only.

docs: include all licenses for vendored packages in the root LICENSE file #1626

Merged
tswast merged 5 commits into from
Apr 17, 2025
Merged

docs: include all licenses for vendored packages in the root LICENSE file #1626

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b079d14
chore: combine all licenses into one file
tswast Apr 17, 2025
f708d83
add tpc-h eula
tswast Apr 17, 2025
d8eaa8e
remove eula from license, not used in main package only benchmarking
tswast Apr 17, 2025
7bc7b0d
remove licences from packages only used for benchmarking
tswast Apr 17, 2025
8300762
exclude from owlbot
tswast Apr 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
exclude from owlbot
  • Loading branch information
@tswast
tswast committed Apr 17, 2025
commit 8300762f3fb677ed817be6a195633db4843ec9d6
2 changes: 2 additions & 0 deletions owlbot.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@
s.move(
templated_files,
excludes=[
# Need a combined LICENSE for all vendored packages.
"LICENSE",

@parthea parthea Apr 17, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a heads up: LICENSE is no longer part of synthtool templates. We may need to update the post processor image if owlbot is reverting changes.

https://github.com/googleapis/synthtool/tree/master/synthtool/gcp/templates/python_library

@tswast tswast Apr 17, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Got it. Yeah, next image update we can remove this line.

# Multi-processing note isn't relevant, as bigframes is responsible for
# creating clients, not the end user.
"docs/multiprocessing.rst",
Expand Down