Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

quietly allow underscores in API names#147

Merged
tangiel merged 1 commit intomasterfrom
underscore
Jun 11, 2018
Merged

quietly allow underscores in API names#147
tangiel merged 1 commit intomasterfrom
underscore

Conversation

@tangiel
Copy link
Contributor

@tangiel tangiel commented Jun 8, 2018

The official documentation for a long time has disallowed underscores
in API names, so I'm leaving it as such. However, for extremely old
users of the old frameworks, this limitation was not in
place. Therefore, I'm removing it from the API validation, but leaving
the @Api documentation in order to not encourage the style.

The official documentation for a long time has disallowed underscores
in API names, so I'm leaving it as such. However, for extremely old
users of the old frameworks, this limitation was not in
place. Therefore, I'm removing it from the API validation, but leaving
the @Api documentation in order to not encourage the style.
@tangiel tangiel requested review from inklesspen and kryzthov June 8, 2018 22:20
@codecov-io
Copy link

codecov-io commented Jun 8, 2018

Codecov Report

Merging #147 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #147   +/-   ##
=========================================
  Coverage     80.01%   80.01%           
  Complexity     1678     1678           
=========================================
  Files           156      156           
  Lines          5595     5595           
  Branches        730      730           
=========================================
  Hits           4477     4477           
  Misses          840      840           
  Partials        278      278
Impacted Files Coverage Δ Complexity Δ
...rver/spi/config/validation/ApiConfigValidator.java 76.57% <ø> (ø) 65 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d41d772...126a136. Read the comment docs.

@tangiel tangiel merged commit 6aa3b8c into master Jun 11, 2018
@tangiel tangiel deleted the underscore branch June 11, 2018 20:39
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.

3 participants