-
Notifications
You must be signed in to change notification settings - Fork 6
feature: Add a new endpoint for metamodels of view types. #121
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
base: main
Are you sure you want to change the base?
Conversation
Clients make a `GET` request on the path `/vsum/views/types/metamodel`, and need to provide the view type as header. Servers respond with a JSON serialization of an `EPackage`. Implemented in the server with the ViewTypeMetamodelEndpoint, and in the client.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
checkstyle
Missing a Javadoc comment.
Line 3 in d24f429
| public record PathEndpointCollector( |
'member def modifier' has incorrect indentation level 4, expected level should be 2.
Line 20 in d24f429
| private PathEndpointCollector endpoints; |
'ctor def modifier' has incorrect indentation level 4, expected level should be 2.
Line 22 in d24f429
| RequestHandler(PathEndpointCollector endpoints) { |
Line is longer than 100 characters (found 114).
Line 12 in d24f429
| public VitruvJavaHttpServer(String host, int port, List<PathEndpointCollector> endpoints) throws IOException { |
'ctor def modifier' has incorrect indentation level 4, expected level should be 2.
Line 12 in d24f429
| public VitruvJavaHttpServer(String host, int port, List<PathEndpointCollector> endpoints) throws IOException { |
Missing a Javadoc comment.
Line 12 in d24f429
| public VitruvJavaHttpServer(String host, int port, List<PathEndpointCollector> endpoints) throws IOException { |
Extra separation in import group before 'org.eclipse.emf.ecore.EPackage'
Line 20 in d24f429
| import org.eclipse.emf.ecore.EPackage; |
Extra separation in import group before 'com.fasterxml.jackson.core.JsonProcessingException'
Line 23 in d24f429
| import com.fasterxml.jackson.core.JsonProcessingException; |
Wrong lexicographical order for 'com.fasterxml.jackson.core.JsonProcessingException' import. Should be before 'org.eclipse.emf.ecore.resource.ResourceSet'.
Line 23 in d24f429
| import com.fasterxml.jackson.core.JsonProcessingException; |
'}' at column 5 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally).
| @@ -1,62 +1,65 @@ | |||
| package tools.vitruv.framework.remote.server.http.java; | |||
|
|
|||
| import static java.net.HttpURLConnection.*; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using the '.' form of import should be avoided - java.net.HttpURLConnection..
| this.endpoints = endpoints; | ||
| } | ||
| RequestHandler(PathEndpointCollector endpoints) { | ||
| this.endpoints = endpoints; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'ctor def' child has incorrect indentation level 6, expected level should be 4.
|
|
@HansMartinA Should I close this PR? |


Clients make a
GETrequest on the path/vsum/views/types/metamodel, and need to provide the view type as header.Servers respond with a JSON serialization of an
EPackage.Implemented in the server with the ViewTypeMetamodelEndpoint, and in the client.