-
Notifications
You must be signed in to change notification settings - Fork 54
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
File Format for DDLS (CDS Data Definitions) #160
Conversation
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.
Probably one point has to be addressed by us.
file-formats/ddls/examples/z_aff_example_ddls.ddls.objectdependencies.json
Show resolved
Hide resolved
A: as a generic issue, we need to consider what the filename extensions should be, I dont think ".cds" is currently used anywhere? |
|
dc803d1
to
9dedd56
Compare
The |
abapGit, yes, it uses baseinfo today: https://github.com/abapGit/abapGit/blob/main/src/objects/zcl_abapgit_object_ddls.clas.abap#L236 |
A: Currently,
|
C: its a generic issue, personally I'd like fewer files, everything will show up as changes anyhow |
Let's look how ABAP classes do it. ADT uses |
Then, most probably, the baseinfo is not related to #131. |
If possible, I personally would also prefer something like In ADT the file extensions have to be unique per object type to open the correct editor. Therefore, we were not able to use just @BeckerWdf Which file extensions would you use for access controls (DCLS), metadata extension (DDLX), and so on? Would it be the same as for DDLS or a different, because we need other syntax colouring, parsing etc...? |
Yes we need another parser but still the language is cds. We would have:
So we know if we need to utilise the ddls, dcls or ddlx parser. And there may be other cds implementations that can understand ddls, dcls and ddlx in one parser. |
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.
Looks good
The https://cap.cloud.sap/docs/cds/ is the "Language Reference Documentation" for CDS? But is this the same CDS we are looking for? If |
So what would you prefer? |
I dont really have a good overview Is ABAP flavored CDS a subset or superset of the other CDS? But |
Let's merge this PR and discuss this in #161 |
Formatting the (.objectdependencies.)json would increase the readability. |
No description provided.