Skip to content

Conversation

@seanbright
Copy link
Contributor

@seanbright seanbright commented Jan 5, 2026

This extends 'channel request hangup' to accept multiple channel names, a POSIX Extended Regular Expression, a glob-like pattern, or a combination of all of them.

UserNote: The 'channel request hangup' CLI command now accepts multiple channel names, POSIX Extended Regular Expressions, glob-like patterns, or a combination of all of them. See the CLI command 'core show help channel request hangup' for full details.

@seanbright
Copy link
Contributor Author

cherry-pick-to: none

@InterLinked1
Copy link
Contributor

cherry-pick-to: none

I'm not seeing how it would break current functionality, how come we can't cherry pick to existing versions?

@seanbright
Copy link
Contributor Author

cherry-pick-to: none

I'm not seeing how it would break current functionality, how come we can't cherry pick to existing versions?

Just being conservative. I'll let others decide.

@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Workflow PRCheck failed
master-pjs1: FAILED TEST: channels/pjsip/geolocation/incoming/basic
master-pjs1: FAILED TEST: channels/pjsip/geolocation/incoming/profile_precedence

@gtjoseph gtjoseph added the recheckpr Trigger PR recheck label Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Workflow PRCheck completed successfully

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Workflow PRCheck completed successfully

@gtjoseph gtjoseph dismissed their stale review January 6, 2026 12:24

Changes made.

Copy link
Member

@gtjoseph gtjoseph left a comment

Choose a reason for hiding this comment

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

Are you going to update ast_complete_channel() as well?

@seanbright
Copy link
Contributor Author

Are you going to update ast_complete_channel() as well?

You mean ast_complete_channels()? Update it how?

@gtjoseph
Copy link
Member

gtjoseph commented Jan 8, 2026

Are you going to update ast_complete_channel() as well?

You mean ast_complete_channels()? Update it how?

So that if you enter a regex or glob and press Tab, it expands to all the matching channels.

@seanbright
Copy link
Contributor Author

Are you going to update ast_complete_channel() as well?

You mean ast_complete_channels()? Update it how?

So that if you enter a regex or glob and press Tab, it expands to all the matching channels.

Does your shell do that? I have never seen that happen in dash or bash.

@gtjoseph
Copy link
Member

gtjoseph commented Jan 8, 2026

Are you going to update ast_complete_channel() as well?

You mean ast_complete_channels()? Update it how?

So that if you enter a regex or glob and press Tab, it expands to all the matching channels.

Does your shell do that? I have never seen that happen in dash or bash.

No but the CLI does for non-regex or non-glob partial entries.

@seanbright
Copy link
Contributor Author

Are you going to update ast_complete_channel() as well?
...
Does your shell do that? I have never seen that happen in dash or bash.

No but the CLI does for non-regex or non-glob partial entries.

No, I am not going to make that change.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Workflow PRCheck completed successfully

This extends 'channel request hangup' to accept multiple channel
names, a POSIX Extended Regular Expression, a glob-like pattern, or a
combination of all of them.

UserNote: The 'channel request hangup' CLI command now accepts
multiple channel names, POSIX Extended Regular Expressions, glob-like
patterns, or a combination of all of them. See the CLI command 'core
show help channel request hangup' for full details.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Attention! This pull request may contain issues that could prevent it from being accepted. Please review the checklist below and take the recommended action. If you believe any of these are not applicable, just add a comment and let us know.

  • The PR description and/or commit message has malformed UserNote, UpgradeNote and/or DeveloperNote trailers. The UserNote, UpgradeNote and DeveloperNote keywords MUST be predeeded by a blank line and followed immediately by a colon and a space before the actual note text. This is to ensure that the note is properly formatted and displayed in the release change logs.

Documentation:

@github-actions github-actions bot added the has-pr-checklist A PR Checklist is present on the PR label Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Workflow PRCheck completed successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-pr-checklist A PR Checklist is present on the PR pr-submit-tests-passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants