Skip to content

fix(google-crc32c): license formatting in pyproject.toml - #18967

Merged
parthea merged 2 commits into
googleapis:mainfrom
APN-Pucky:patch-1
Nov 4, 2026
Merged

parthea merged 2 commits into
googleapis:mainfrom
APN-Pucky:patch-1

Conversation

@APN-Pucky

Copy link
Copy Markdown
Contributor

Gentoo QA complains:

 * QA Notice: setuptools warnings detected:
 * 
 * 	`extras_require` overwritten in `pyproject.toml` (optional-dependencies)
 * 	`project.license` as a TOML table is deprecated

Duplicate of #18950 that got automatically closed after I git cloned my patch with depth 1 and tried to amend the commit, resulting in squashing the whole repo...

@APN-Pucky
APN-Pucky requested a review from a team as a code owner November 3, 2026 16:09

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates the license field format in pyproject.toml from a table to a simple string. I have no feedback to provide.

@parthea parthea left a comment

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.

Thanks @APN-Pucky!

@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Nov 3, 2026
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Nov 3, 2026
ohmayr added a commit that referenced this pull request Nov 4, 2026
This PR is needed to unblock
#18967 where the
windows build is failing with

```
ERROR: Exception:
Traceback (most recent call last):
  File "C:\Program Files\Python310\lib\site-packages\pip\_internal\cli\base_command.py", line 110, in _run_wrapper
    status = _inner_run()
  File "C:\Program Files\Python310\lib\site-packages\pip\_internal\cli\base_command.py", line 103, in _inner_run
    return self.run(options, args)
  File "C:\Program Files\Python310\lib\site-packages\pip\_internal\cli\req_command.py", line 162, in wrapper
    return func(self, options, args)
  File "C:\Program Files\Python310\lib\site-packages\pip\_internal\commands\install.py", line 729, in run
    _eagerly_import_modules(module_names)
  File "C:\Program Files\Python310\lib\site-packages\pip\_internal\commands\install.py", line 176, in _eagerly_import_modules
    __import__(module)
  File "C:\Program Files\Python310\lib\site-packages\readline.py", line 34, in <module>
    rl = Readline()
  File "C:\Program Files\Python310\lib\site-packages\pyreadline\rlmain.py", line 422, in __init__
    BaseReadline.__init__(self)
  File "C:\Program Files\Python310\lib\site-packages\pyreadline\rlmain.py", line 62, in __init__
    mode.init_editing_mode(None)
  File "C:\Program Files\Python310\lib\site-packages\pyreadline\modes\emacs.py", line 633, in init_editing_mode
    self._bind_key('space',       self.self_insert)
  File "C:\Program Files\Python310\lib\site-packages\pyreadline\modes\basemode.py", line 162, in _bind_key
    if not callable(func):
  File "C:\Program Files\Python310\lib\site-packages\pyreadline\py3k_compat.py", line 8, in callable
    return isinstance(x, collections.Callable)
AttributeError: module 'collections' has no attribute 'Callable'
```

I filed #18972
for the follow up issue to audit builds that are still using Python 3.10
and upgrade them

Co-authored-by: ohmayr <omairn@google.com>
@parthea parthea added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Nov 4, 2026
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Nov 4, 2026
@parthea
parthea merged commit 6cb0223 into googleapis:main Nov 4, 2026
119 checks passed
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.

4 participants