Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Remove outdated TODO #149

Merged
merged 1 commit into from
Jun 17, 2021
Merged

Conversation

xtuchyna
Copy link
Member

@xtuchyna xtuchyna commented Jun 17, 2021

Related Issues and Dependencies

Closes #60

This introduces a breaking change

  • Yes
  • No

@sesheta sesheta requested review from harshad16 and KPostOffice June 17, 2021 13:48
@sesheta sesheta added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 17, 2021
@xtuchyna xtuchyna changed the title Remove irrelavant TODO Remove outdated TODO Jun 17, 2021
@goern
Copy link
Member

goern commented Jun 17, 2021

/approve

@sesheta
Copy link
Member

sesheta commented Jun 17, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: goern

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 17, 2021
@sesheta sesheta merged commit 5019025 into thoth-station:master Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

use the return value more efficiently to assign only active managers
3 participants