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

Fix Typo in Notification Hook Docs #917

Merged
merged 5 commits into from
Jan 7, 2022
Merged

Conversation

J12934
Copy link
Member

@J12934 J12934 commented Jan 7, 2022

Noticed a small mistake in the notification hook docs.

J12934 and others added 2 commits January 7, 2022 10:48
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
@J12934 J12934 added the documentation Improvements or additions to documentation label Jan 7, 2022
@J12934 J12934 self-assigned this Jan 7, 2022
@J12934 J12934 added this to In progress in secureCodeBox v3 via automation Jan 7, 2022
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 moved this from In progress to To Review in secureCodeBox v3 Jan 7, 2022
secureCodeBox v3 automation moved this from To Review to Reviewer approved Jan 7, 2022
malexmave
malexmave previously approved these changes Jan 7, 2022
@J12934 J12934 enabled auto-merge January 7, 2022 09:53
@J12934 J12934 disabled auto-merge January 7, 2022 09:56
@J12934
Copy link
Member Author

J12934 commented Jan 7, 2022

Just noticed this snippet is even more wrong than I initially thought 😅

Also made the indentation more consistent in the other example

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
secureCodeBox v3 automation moved this from Reviewer approved to To Review Jan 7, 2022
Signed-off-by: GitHub Actions <securecodebox@iteratec.com>
secureCodeBox v3 automation moved this from To Review to Reviewer approved Jan 7, 2022
@J12934 J12934 merged commit 523e60c into main Jan 7, 2022
@J12934 J12934 deleted the docs/notification-hook-fix branch January 7, 2022 12:01
secureCodeBox v3 automation moved this from Reviewer approved to Done Jan 7, 2022
@J12934 J12934 moved this from Done to counter in secureCodeBox v3 Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants