exercism / abap

Exercism exercises in ABAP.
https://exercism.org/tracks/abap
MIT License
72 stars 36 forks source link

Known Issues #202

Open larshp opened 1 year ago

larshp commented 1 year ago

Using AT

https://github.com/exercism/abap/issues/134 https://github.com/exercism/abap/issues/201

=> https://github.com/abaplint/transpiler/issues/706

FOR loops with identical declared variable names

https://github.com/exercism/abap/issues/194 https://github.com/exercism/abap/issues/200 https://github.com/exercism/abap/issues/261

=> first part of solution is implementing renaming of variables in abaplint

SELECT FROM @itab

note that this will probably not be supported on Exercism, as a running database will be required

https://github.com/exercism/abap/issues/181

https://github.com/abaplint/transpiler/issues/669

BaerbelW commented 1 year ago

Hi Lars - how about mentioning these known issues in the relevant descriptions of the Code Challenges where they apply? It could help prevent frustations and false starts if people know that for example AT-constructs won't currently work on Exercism (even though solving the Aggregation challenge would be quite straightforward - I think - if it worked).

Cheers Bärbel

larshp commented 1 year ago

yea, well, I guess many exercises can potentially be solved using AT or FOR

also, when exercises are updated, everyone will get a suggestion to update the exercise to latest, so would like to avoid updating them too often, image

Instead of spending my time to update documentation, I can spend it to solve the actual issues...

BaerbelW commented 1 year ago

Good points, Lars! I wasn't aware that just updating the description would trigger the update notice for users.

Cheers Bärbel

larshp commented 1 year ago

VALUE in DATA: will be fixed with https://github.com/exercism/abap-test-runner/pull/168 when approved by @exercism/maintainers-admin

larshp commented 1 year ago

VALUE in DATA parts of CLASS DEFINITION are ignore: fixed, removed from list

larshp commented 1 year ago

Using AT in LOOPs should now work, I'll keep it in the list until the corresponding issues are confirmed worknig