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

Replaces executor with adapter in places we missed #63

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

skrawcz
Copy link
Collaborator

@skrawcz skrawcz commented Feb 7, 2022

We missed a few spots. This changes executor to adapter
to be inline with the terminology that we want to use
in the code base.

Changes

  • replaces executor with adapter in places we missed.

Testing

  1. unit testing FTW

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows the standards laid out in the TODO link to standards
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Python

  • python 3.6
  • python 3.7

We missed a few spots. This changes executor to adapter
to be inline with the terminology that we want to use
in the code base.
@skrawcz skrawcz merged commit 8289b9a into main Feb 10, 2022
@skrawcz skrawcz deleted the fix_graph_var_name branch February 10, 2022 00:06
gitbook-com bot pushed a commit that referenced this pull request Feb 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants