Skip to content

Conversation

@Lettly
Copy link
Contributor

@Lettly Lettly commented Jun 30, 2025

This pull request updates the documentation in docs/scenarios/actions.rst to improve clarity and accuracy. The most notable changes include adding the nop command to the list of commands that can execute actions and fixing a typo in the description of the log action.

Documentation improvements:

  • Added nop to the list of commands capable of executing actions, alongside recv and recvCmd, to ensure completeness.
  • Corrected a typographical error in the description of the log action by fixing "aa log file" to "a log file."

@Lettly Lettly changed the title docs: Update actions.rst by adding support for nop command docs: update actions.rst by adding support for nop command Jun 30, 2025
@orgads orgads requested a review from Copilot July 11, 2025 10:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the documentation in docs/scenarios/actions.rst to include the nop command among those that can execute actions and fixes a typo in the log action description.

  • Added nop to the list of commands capable of executing actions
  • Corrected a typographical error in the log action description
Comments suppressed due to low confidence (2)

docs/scenarios/actions.rst:4

  • [nitpick] Consider adding a comma before 'or' in the list for consistent punctuation (e.g., recv, recvCmd, or nop).
In a `recv`, `recvCmd` or `nop` command, you have the possibility to execute

docs/scenarios/actions.rst:4

  • [nitpick] Consider ordering the commands alphabetically (e.g., nop, recv, recvCmd) for easier reference.
In a `recv`, `recvCmd` or `nop` command, you have the possibility to execute

@orgads orgads merged commit 14b0691 into SIPp:master Jul 11, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants