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

Invalid abapdoc? #404

Open
larshp opened this issue Oct 11, 2022 · 5 comments
Open

Invalid abapdoc? #404

larshp opened this issue Oct 11, 2022 · 5 comments
Assignees

Comments

@larshp
Copy link
Collaborator

larshp commented Oct 11, 2022

Hi,
When activating abapGit standalone, the editor gives the following warnings regarding the abapdoc,

ref https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abendoccomment.htm

750 SP0022 and 750 SP0006

seems okay on 754 SP0002

https://help.sap.com/doc/abapdocu_754_index_htm/7.54/en-US/abennews-752-abap_doc.htm

guess there is a SAP note to downport the logic to 750 and 702?

image

@larshp
Copy link
Collaborator Author

larshp commented Oct 13, 2022

Actions:

  1. strip abapdoc comments in https://github.com/abapGit/aff_mirror, perhaps replace with a reference to this repository
  2. consider if its worth changing https://github.com/SAP/abap-file-formats/blob/main/abaplint.jsonc#L13
  3. @schneidermic0 to check for SAP Notes

@albertmink
Copy link
Contributor

albertmink commented Oct 13, 2022

Maybe I can take oder the deletion of AbapDoc in the mirror repository. Some 'sed' magic should do it

@larshp
Copy link
Collaborator Author

larshp commented Oct 13, 2022

@albertmink yea, cool, sed solves many things 😄 if it solves too much, then abaplint will complain

@albertmink
Copy link
Contributor

Action 1 is implemented, see abapGit/aff_mirror#53

@schneidermic0 schneidermic0 self-assigned this Oct 14, 2022
@schneidermic0
Copy link
Contributor

schneidermic0 commented Oct 14, 2022

@schneidermic0 to check for SAP Notes

I will check whether there is a SAP Note.

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

No branches or pull requests

3 participants