-
Notifications
You must be signed in to change notification settings - Fork 12
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
Statistics: Commented lines: Distinguish commented code from comments #73
Comments
suggest pushing the history to git |
You mean... starting from the first version and artificially creating a commit on each new change? That could work, yes. But this would create a complex dependency on an external system. I like it that today For example, my current client has old and established habits and in order to introduce something new, that something must be proven to be simple, risk free and easy to understand by those who will both approve it and use it. |
well, if the client runs SAP, the company is typically of a decent size. I'd assume all companies of decent size already use git in their IT departments |
Maybe. But my experience tells me that in most of these big organisations, this kind of changes is not always easy to implement (even if technically is very simple). Regardless, if I were to do it, it would already have to be an optional advanced setting, because I really like it that |
yea 👍 organizations are difficult |
I could use something similar to what abapOpenChecks is doing in
SCAN ABAP-SOURCE it_code
TOKENS INTO lt_tokens
STATEMENTS INTO lt_statements
WITH ANALYSIS.
IF sy-subrc <> 0.
rs_result-match = abap_false.
RETURN.
ENDIF. But, since I want to find out how many commented lines are actually ABAP code, it seems that I need to send it one ABAP command at a time (which may be comprised of multiple lines) otherwise it returns false if at least one line is not valid ABAP code. I still didn't figure out a simple way to do this. |
No description provided.
The text was updated successfully, but these errors were encountered: