Skip to content

Removing unused _EMPTY sentinel from base Connection.#643

Merged
tseaver merged 1 commit intogoogleapis:masterfrom
dhermes:remove-unused-empty
Feb 14, 2015
Merged

Removing unused _EMPTY sentinel from base Connection.#643
tseaver merged 1 commit intogoogleapis:masterfrom
dhermes:remove-unused-empty

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Feb 14, 2015

I noticed this during the discussion in #639.

This change is borderline not requiring a PR, but I figured it couldn't hurt as an FYI.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 14, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 73650fd on dhermes:remove-unused-empty into c6718f3 on GoogleCloudPlatform:master.

tseaver added a commit that referenced this pull request Feb 14, 2015
Removing unused _EMPTY sentinel from base Connection.
@tseaver tseaver merged commit bf3507f into googleapis:master Feb 14, 2015
@dhermes dhermes deleted the remove-unused-empty branch February 14, 2015 01:49
vchudnov-g pushed a commit that referenced this pull request Sep 20, 2023
Source-Link: https://togithub.com/googleapis/synthtool/commit/909573ce9da2819eeb835909c795d29aea5c724e
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:ddf4551385d566771dc713090feb7b4c1164fb8a698fe52bbe7670b24236565b
parthea pushed a commit that referenced this pull request Nov 24, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: core cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants