Skip to content

added /chapters NestBot command#410

Merged
arkid15r merged 5 commits into
OWASP:mainfrom
nitinawari:slackbot/chapters-command
Jan 11, 2025
Merged

added /chapters NestBot command#410
arkid15r merged 5 commits into
OWASP:mainfrom
nitinawari:slackbot/chapters-command

Conversation

@nitinawari
Copy link
Copy Markdown
Contributor

@nitinawari nitinawari commented Jan 10, 2025

close #360

checklist

  • added chapter command
  • added --help flag
  • added search term and region as argument to help flag
  • added /chapters command to /owasp --help as argument
  • added test cases

@nitinawari nitinawari requested a review from arkid15r as a code owner January 10, 2025 13:54
@nitinawari nitinawari requested a review from kasya as a code owner January 10, 2025 18:26
@nitinawari nitinawari force-pushed the slackbot/chapters-command branch from 336677a to 628e402 Compare January 10, 2025 19:01
@arkid15r arkid15r changed the title added /chapter NestBot command added /chapters NestBot command Jan 10, 2025
Copy link
Copy Markdown
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

Please take care of the conflicts and the unrelated code and I'll review it.

@nitinawari nitinawari force-pushed the slackbot/chapters-command branch 2 times, most recently from 6dc4a66 to 628e402 Compare January 11, 2025 07:37
@nitinawari nitinawari force-pushed the slackbot/chapters-command branch from 628e402 to ac1e3a4 Compare January 11, 2025 11:24
@nitinawari nitinawari requested a review from arkid15r January 11, 2025 11:38
@arkid15r arkid15r enabled auto-merge January 11, 2025 19:51
Copy link
Copy Markdown
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

Merging this in, thanks @nitinawari

@arkid15r arkid15r added this pull request to the merge queue Jan 11, 2025
Merged via the queue into OWASP:main with commit be41fd2 Jan 11, 2025
@nitinawari nitinawari deleted the slackbot/chapters-command branch January 12, 2025 05:44
shdwcodr pushed a commit to shdwcodr/Nest that referenced this pull request Jun 5, 2025
* added /chapter NestBot command

* pre-commit fix

* added /chapter command to /owasp --help as argument

* pre-commit

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement NestBot /chapters command

2 participants