Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

extsvc: remove unused vars #55660

Merged
merged 1 commit into from
Aug 9, 2023
Merged

extsvc: remove unused vars #55660

merged 1 commit into from
Aug 9, 2023

Conversation

@mrnugget
Copy link
Contributor

@mrnugget mrnugget commented Aug 9, 2023

The linter in #55647 flagged these. They are unused and are a left-over after the migration to the "Variant" types in this file.

Test plan

  • Existing CI
@mrnugget
extsvc: remove unused vars
2e0de75 
The linter in #55647 flagged these. They are unused and are a left-over after the migration to the "Variant" types in this file.
@mrnugget mrnugget requested a review from a team August 9, 2023 07:05
@cla-bot cla-bot bot added the cla-signed label Aug 9, 2023
@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Aug 9, 2023

Codenotify: Notifying subscribers in CODENOTIFY files for diff 4ffba4e...2e0de75.

Notify File(s)
@eseliger internal/extsvc/types.go
@sashaostrikov internal/extsvc/types.go
@mrnugget mrnugget merged commit ec8c788 into main Aug 9, 2023
@mrnugget mrnugget deleted the mrn/remove-unused-vars branch August 9, 2023 08:38
davejrt pushed a commit that referenced this pull request Aug 9, 2023
@mrnugget @davejrt
extsvc: remove unused vars (#55660)
de113c4 
The linter in #55647 flagged these. They are unused and are a left-over after the migration to the "Variant" types in this file. ## Test plan - Existing CI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed

5 participants

@mrnugget @sourcegraph-bot @burmudar @sashaostrikov