Skip to content

Disable info-contact rule for initial CAMARA API specification#367

Merged
rartych merged 2 commits intocamaraproject:mainfrom
ravindrapalaskar17:Update-info-contact-spectral-rule
Jan 9, 2025
Merged

Disable info-contact rule for initial CAMARA API specification#367
rartych merged 2 commits intocamaraproject:mainfrom
ravindrapalaskar17:Update-info-contact-spectral-rule

Conversation

@ravindrapalaskar17
Copy link
Contributor

What type of PR is this?

  • correction

What this PR does / why we need it:

This PR disable the info-contact rule.

Which issue(s) this PR fixes:

Fixes #

As discussed, and decided in the CAMARA Commonalities call, we need to disable the info-contact spectral rule because we are preparing the first version of the CAMARA API specification, where contact information is not required. After the API specification is finalized, the developers/Organization responsible for this API will add their contact details.
As per CAMARA standards

Changelog input:

30.12.2024: Updated info-contact rule

* Update info-contact rule

* Update .spectral.yml
PedroDiez
PedroDiez previously approved these changes Jan 8, 2025
Copy link
Contributor

@PedroDiez PedroDiez left a comment

Choose a reason for hiding this comment

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

LGTM

patrice-conil
patrice-conil previously approved these changes Jan 8, 2025
Copy link
Contributor

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

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

LGTM

@rartych rartych dismissed stale reviews from patrice-conil and PedroDiez via e85520e January 8, 2025 15:55
@rartych
Copy link
Contributor

rartych commented Jan 8, 2025

Conflict with the changelog in the file header resolved

@rartych rartych requested a review from PedroDiez January 8, 2025 15:58
Copy link
Contributor

@PedroDiez PedroDiez left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants