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.

Sign up for GitHub

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

Jump to bottom

[CTOR-179] Include orthograph/syntax checks in CI for MC documentation #4788

Merged
merged 36 commits into from
Feb 1, 2024
Merged

[CTOR-179] Include orthograph/syntax checks in CI for MC documentation #4788

merged 36 commits into from
Feb 1, 2024

Conversation

lucie-dubrunfaut
Copy link
Contributor

@lucie-dubrunfaut lucie-dubrunfaut commented Dec 21, 2023 •
edited
Loading

Description

Include orthograph/syntax checks in CI for MC documentation using perl library to check the syntax of each help message in the added/modified files.

Fixes # (issue)

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

@lucie-dubrunfaut lucie-dubrunfaut changed the title Add pod_spell_check test and githubaction [CTOR-179] Include orthograph/syntax checks in CI for MC documentation Dec 21, 2023
@lucie-dubrunfaut lucie-dubrunfaut marked this pull request as ready for review December 22, 2023 10:40
@lucie-dubrunfaut lucie-dubrunfaut requested a review from a team as a code owner December 22, 2023 10:40
@lucie-dubrunfaut @kduret
Update .github/workflows/spellchecker.yml
ad558d8 
Co-authored-by: Kevin Duret <kduret@centreon.com>
kduret
kduret previously approved these changes Jan 3, 2024
View reviewed changes
sc979
sc979 previously approved these changes Jan 3, 2024
View reviewed changes
@lucie-dubrunfaut lucie-dubrunfaut dismissed stale reviews from sc979 and kduret via 4eab96e January 16, 2024 14:11
@lucie-dubrunfaut lucie-dubrunfaut requested a review from a team as a code owner January 16, 2024 14:11
@sdepassio sdepassio added the do-not-spellcheck avoid running the spellchecks in github action label Jan 17, 2024
@lucie-dubrunfaut lucie-dubrunfaut removed the do-not-spellcheck avoid running the spellchecks in github action label Jan 17, 2024
sdepassio and others added 5 commits January 17, 2024 17:37
@sdepassio
test
8f0ae7b
Remove trashs
9e1b373
Merge branch 'CTOR-179-include-orthograph-syntax-checks-in-ci-for-mc-…
e9ec918 
…documentation' of github.com:centreon/centreon-plugins into CTOR-179-include-orthograph-syntax-checks-in-ci-for-mc-documentation
Handle with stopword path
d474fc6
Update stopwords.t
b215026
@lucie-dubrunfaut lucie-dubrunfaut merged commit ce2d461 into develop Feb 1, 2024
12 checks passed
@lucie-dubrunfaut lucie-dubrunfaut deleted the CTOR-179-include-orthograph-syntax-checks-in-ci-for-mc-documentation branch February 1, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sc979 sc979 sc979 approved these changes

@sdepassio sdepassio sdepassio approved these changes

@Evan-Adam Evan-Adam Evan-Adam approved these changes

@kduret kduret kduret left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lucie-dubrunfaut @kduret @sc979 @sdepassio @Evan-Adam