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

Small nits to show queue and change ack#8

Open
jaimejim wants to merge 1 commit into
mnot:mainfrom
jaimejim:patch-2
Open

Small nits to show queue and change ack#8
jaimejim wants to merge 1 commit into
mnot:mainfrom
jaimejim:patch-2

Conversation

@jaimejim

Copy link
Copy Markdown
Contributor

We found it more useful to print the current queue after q+ as well.
As ack is commonly used, we could instead use qack or something else.

We found it more useful to print the current queue after q+ as well.
As `ack` is commonly used, we could instead use `qack` or something else.
@mnot

mnot commented Apr 2, 2020

Copy link
Copy Markdown
Owner

Hey, thanks for the contribution.

I don't think it's good to show the queue on every q+; that can get disruptive if there are a number of people.

With regard to ack, I here you; still thinking about whether it's necessary. My model for this is the behaviour on Zakim (linked from README), where they use ack without an issue.

Base automatically changed from master to main February 11, 2021 01:31
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