abapTimeMachine icon indicating copy to clipboard operation
abapTimeMachine copied to clipboard

Make code compatible with v740sp02

Open nununo opened this issue 3 years ago • 5 comments

Consider this report: https://abaplint.app/stats/abapinho/abapTimeMachine/statement_compatibility

As suggested by Lars here: https://github.com/abapinho/abapTimeMachine/issues/86#issuecomment-874086296

nununo avatar Jul 05 '21 14:07 nununo

Out of curiousity I did a local check to see what kind of statements need to be changed for 740sp02, it's mostly "quality of life" stuff:

  • inline data
  • SWITCH, FOR, VALUE, etc. statments

Could just change the readme to require 740sp08 and call it a day, up to you 😅

g-back avatar Jul 06 '21 09:07 g-back

Out of curiousity I did a local check to see what kind of statements need to be changed for 740sp02, it's mostly "quality of life" stuff:

How did you do it @g-back? Did you install abaplint in your fork? I tried to go to abaplint's compatibility chart for your fork but it doesn't seem to be active.

nununo avatar Jul 07 '21 09:07 nununo

1: install Node on the local PC 2: do npm install @abaplint/cli -g this will install abaplint as a global executeable on the system, use the same command to update to latest 3: clone the repo to local PC 4: adjust abaplint.json 5 in the repo folder, run abaplint on command line

larshp avatar Jul 07 '21 09:07 larshp

Personally I used abaplint-sci client and server - but that is a bit more involved to set up than what Lars suggested.

g-back avatar Jul 07 '21 10:07 g-back

heh, yea

alternatively, create a new branch, edit the abaplint.json in the branch, open a draft pull request

larshp avatar Jul 07 '21 11:07 larshp