Skip to content

fix: address Spectral, Python, and GPLint validation warnings - #27

Merged
usmanTheCoder merged 2 commits into
camaraproject:mainfrom
ALIIQBAL786:fix/validation-warnings
Oct 16, 2026
Merged

usmanTheCoder merged 2 commits into
camaraproject:mainfrom
ALIIQBAL786:fix/validation-warnings

Conversation

@ALIIQBAL786

Copy link
Copy Markdown
Contributor

Summary

Addresses the 13 warnings and 4 hints from validation run #38856185530 as requested by @hdamker.

Spectral (S-rules):

  • S-022: Fixed apiRoot server default to http://localhost:9091
  • S-310/S-311: Added format: int32, minimum: 100, maximum: 599 to status integer in ErrorInfo
  • S-312 (×8): Added maxLength to all string properties missing it — lastChecked/reportedDate (29), imei fields (15), code (80), message (256), XCorrelator (256)

Python checks (P-rules):

  • P-013: Removed placeholder README.MD from code/API_definitions/
  • P-026 (×3): Added mandatory info.description template blocks — request-body-strictness, authorization-and-authentication, additional-error-responses
  • P-038: Added externalDocs with repository URL

GPLint (G-rules):

  • G-002: Fixed feature tag indentation to 2 spaces
  • G-022: Removed unused description column from scenario Examples table

- Add mandatory CAMARA info.description template blocks (P-026):
  request-body-strictness, authorization-and-authentication,
  additional-error-responses
- Add externalDocs pointing to repository (P-038)
- Remove placeholder README.MD from code/API_definitions/ (P-013)
- Fix apiRoot server default to http://localhost:9091 (S-022)
- Add maxLength to all string properties without enum/const (S-312):
  lastChecked, reportedDate (29), imei fields (15),
  code (80), message (256), XCorrelator (256)
- Add format: int32 and minimum/maximum to status integer (S-310/S-311)
- Fix feature tag indentation to 2 spaces (G-002)
- Remove unused 'description' column from Examples table (G-022)
@usmanTheCoder
usmanTheCoder merged commit 7b55f0a into camaraproject:main Oct 16, 2026
2 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