Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Place command response in same thread #63

Merged
merged 1 commit into from
Sep 25, 2020
Merged

Place command response in same thread #63

merged 1 commit into from
Sep 25, 2020

Conversation

hanzei
Copy link
Contributor

@hanzei hanzei commented Sep 24, 2020

Summary

Yet another instance of this bug 😕

Ticket Link

Fixes #62

@hanzei hanzei added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Sep 24, 2020
@hanzei hanzei added this to the v0.2.1 milestone Sep 24, 2020
@codecov
Copy link

codecov bot commented Sep 24, 2020

Codecov Report

Merging #63 into master will decrease coverage by 0.08%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   26.11%   26.03%   -0.09%     
==========================================
  Files           7        7              
  Lines         314      315       +1     
==========================================
  Hits           82       82              
- Misses        215      216       +1     
  Partials       17       17              
Impacted Files Coverage Δ
server/command.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ad30fb...379f05a. Read the comment docs.

@jfrerich jfrerich removed the 2: Dev Review Requires review by a core committer label Sep 25, 2020
@hanzei hanzei mentioned this pull request Sep 25, 2020
4 tasks
@hanzei
Copy link
Contributor Author

hanzei commented Sep 25, 2020

Given that the fix is straight forward and I've tested it myself, I'm going to merge this PR without QA review.

@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Sep 25, 2020
@hanzei hanzei merged commit 1949969 into master Sep 25, 2020
@hanzei hanzei deleted the 62_rhs branch September 25, 2020 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Queuing an item from the RHS does not put it in a thread
3 participants