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

Model chat id clarification #3742

Merged
merged 1 commit into from
Jun 25, 2021
Merged

Conversation

EricMichaelSmith
Copy link
Contributor

Patch description
Clarify a comment the model_chat crowdsourcing task to no longer imply that the worker may not have an id set, because we manually always set the worker's id to 'Worker'

Testing steps
CI checks

@EricMichaelSmith EricMichaelSmith changed the title Clarification Model chat id clarification Jun 23, 2021
Copy link
Contributor

@klshuster klshuster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

failing tests look transient

@EricMichaelSmith EricMichaelSmith merged commit a88a179 into master Jun 25, 2021
@EricMichaelSmith EricMichaelSmith deleted the clarification-model-chat branch June 25, 2021 15:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants