Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

implement OSPO guidelines #496

Merged
merged 6 commits into from
Mar 4, 2024
Merged

implement OSPO guidelines #496

merged 6 commits into from
Mar 4, 2024

Conversation

sbasan
Copy link
Collaborator

@sbasan sbasan commented Feb 27, 2024

Pull Request summary:

Fix issues after OSPO review

Description of changes:

  1. Include copyright notice in each python source file
  2. Add code guidelines section link in CONTRIBUTING.md
  3. Fix catalystwan/models/configuration/feature_profile/README.md
  4. Fix catalystwan/api/templates/README.md
  5. Apply inclusive language policy when possible.
    • bumped version and regenerated ENDPOINTS.md to include renamed items
    • some strings cannot be removed as they're defined externally eg.:
      GET /template/policy/list/urlwhitelist
      https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information

Checklist:

  • Make sure to run pre-commit before committing changes
  • Make sure all checks have passed
  • PR description is clear and comprehensive
  • Mentioned the issue that this PR solves (if applicable)
  • Make sure you test the changes

@sbasan sbasan changed the title OSPO compliance implement OSPO guidelines Feb 27, 2024
@sbasan sbasan marked this pull request as ready for review February 27, 2024 15:22
@sbasan sbasan merged commit 23b2d71 into main Mar 4, 2024
12 checks passed
@sbasan sbasan deleted the fix/ospo-review branch March 4, 2024 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants