Skip to content

Commit

Permalink
address review comments, fix maintained libs/repos
Browse files Browse the repository at this point in the history
  • Loading branch information
shorgi committed Oct 20, 2023
1 parent e4b2d56 commit 713e917
Show file tree
Hide file tree
Showing 15 changed files with 18 additions and 17 deletions.
5 changes: 3 additions & 2 deletions implementations/apollo-server/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,17 @@
"repository": {
"name": "apollo-server",
"owner": "apollographql",
"maintainer" : "apollographql",
"link": "https://github.com/apollographql/apollo-server"
},
"coreLibrary": {
"name": "GraphQL.js",
"owner": "apollographql",
"maintainer": "apollographql",
"link": "https://www.npmjs.com/package/graphql"
},
"federationlibrary": {
"name": "Apollo Subgraph",
"owner": "apollographql",
"maintainer": "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 0 additions & 1 deletion implementations/async-graphql/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
"repository": {
"name": "async-graphql",
"owner": "async-graphql",
"maintainer": "apollographql",
"link": "https://github.com/async-graphql/async-graphql"
}
}
1 change: 1 addition & 0 deletions implementations/dgs/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"federationlibrary": {
"name": "federation-jvm",
"owner": "apollographql",
"maintainer" : "apollographql",
"link": "https://github.com/apollographql/federation-jvm"
}
}
1 change: 1 addition & 0 deletions implementations/express-graphql/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/graphql-java-kickstart/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"federationlibrary": {
"name": "federation-jvm",
"owner": "apollographql",
"maintainer": "apollographql",
"link": "https://github.com/apollographql/federation-jvm"
}
}
1 change: 1 addition & 0 deletions implementations/graphql-yoga/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/helix/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/hotchocolate/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
"federationlibrary": {
"name": "federation-hotchocolate",
"owner": "apollographql",
"maintainer": "apollographql",
"link": "https://github.com/apollographql/federation-hotchocolate"
}
}
1 change: 1 addition & 0 deletions implementations/mercurius/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/neo4j-graphql/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/nestjs-code-first/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/nestjs/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
},
"federationlibrary": {
"name": "Apollo Subgraph",
"maintainer" : "apollographql",
"link": "https://www.npmjs.com/package/@apollo/subgraph"
}
}
1 change: 1 addition & 0 deletions implementations/spring-graphql/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"federationlibrary": {
"name": "federation-jvm",
"owner": "apollographql",
"maintainer": "apollographql",
"link": "https://github.com/apollographql/federation-jvm"
}
}
1 change: 0 additions & 1 deletion implementations/strawberry-graphql/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
"name": "strawberry",
"owner": "strawberry-graphql",
"link": "https://github.com/strawberry-graphql/strawberry",
"maintainer": "apollographql"
},
"coreLibrary": {
"name": "GraphQL-core 3",
Expand Down
17 changes: 4 additions & 13 deletions packages/compatibility/src/utils/markdown.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ class MarkdownFile {
| Icon | Description |
| ---- | ---------------------------------------------------- |
| ${apolloIcon} | Library is maintained by Apollo |
| ${apolloIcon} | Maintained by Apollo |
| 🟢 | Functionality is supported |
| ❌ | Critical functionality is NOT supported |
| 🔲 | Additional federation functionality is NOT supported |
Expand Down Expand Up @@ -143,10 +143,7 @@ class MarkdownFile {
let repoName = result.repository.owner
? `${result.repository.owner}/${result.repository.name}`
: result.repository.name;
if (
result.repository.owner === apolloName ||
result.repository.maintainer === apolloName
) {
if (result.repository.maintainer === apolloName) {
repoName += `  ${apolloIcon}`;
}
content += `Github: <a href="${result.repository.link}">${repoName}</a><br/>
Expand All @@ -159,10 +156,7 @@ Last Release: ${lastReleaseDate}<br/><br/>`;
let coreLibraryName = result.coreLibrary.owner
? `${result.coreLibrary.owner}/${result.coreLibrary.name}`
: result.coreLibrary.name;
if (
result.coreLibrary.owner === apolloName ||
result.coreLibrary.maintainer === apolloName
) {
if (result.coreLibrary.maintainer === apolloName) {
coreLibraryName += `&nbsp;&nbsp;${apolloIcon}`;
}
content += `Core Library: <a href="${result.coreLibrary.link}">${coreLibraryName}</a><br/>`;
Expand All @@ -172,10 +166,7 @@ Last Release: ${lastReleaseDate}<br/><br/>`;
let fedLibraryName = result.federationlibrary.owner
? `${result.federationlibrary.owner}/${result.federationlibrary.name}`
: result.federationlibrary.name;
if (
result.federationlibrary.owner === apolloName ||
result.federationlibrary.maintainer === apolloName
) {
if (result.federationlibrary.maintainer === apolloName) {
fedLibraryName += `&nbsp;&nbsp;${apolloIcon}`;
}
content += `Federation Library: <a href="${result.federationlibrary.link}">${fedLibraryName}</a><br/>`;
Expand Down

0 comments on commit 713e917

Please sign in to comment.