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.

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

Update Service APIs references to Gateway APIs #3397

Merged
merged 3 commits into from Feb 24, 2021

Conversation

youngnick
Copy link
Member

Signed-off-by: Nick Young ynick@vmware.com

@youngnick youngnick added this to the 1.13.0 milestone Feb 23, 2021
@youngnick youngnick self-assigned this Feb 23, 2021
@youngnick youngnick requested a review from a team as a code owner February 23, 2021 22:45
@youngnick youngnick requested review from stevesloka and danehans and removed request for a team February 23, 2021 22:45
@codecov
Copy link

codecov bot commented Feb 23, 2021

Codecov Report

Merging #3397 (322ab4f) into main (495a691) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3397      +/-   ##
==========================================
+ Coverage   75.39%   75.42%   +0.03%     
==========================================
  Files          98       98              
  Lines        6588     6588              
==========================================
+ Hits         4967     4969       +2     
+ Misses       1513     1512       -1     
+ Partials      108      107       -1     
Impacted Files Coverage Δ
internal/dag/cache.go 91.51% <0.00%> (+0.60%) ⬆️

@danehans
Copy link
Contributor

@youngnick nit: the project should be referenced in the singular tense (Gateway API) and not plural (Gateway APIs). I think it does make sense to state that Gateway API is a group or collection of APIs or resources so the reader understands that the project is not a single API.

Signed-off-by: Nick Young <ynick@vmware.com>
design/gateway-apis-implementation.md Outdated Show resolved Hide resolved
design/gateway-apis-implementation.md Outdated Show resolved Hide resolved
design/gateway-apis-implementation.md Outdated Show resolved Hide resolved
design/gateway-apis-implementation.md Outdated Show resolved Hide resolved
design/gateway-apis-implementation.md Outdated Show resolved Hide resolved
Signed-off-by: Nick Young <ynick@vmware.com>
@youngnick youngnick merged commit 40c828a into projectcontour:main Feb 24, 2021
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.

None yet

3 participants