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

Known Issues #202

Open
larshp opened this issue Sep 24, 2022 · 6 comments
Open

Known Issues #202

larshp opened this issue Sep 24, 2022 · 6 comments

Comments

@larshp
Copy link
Member

larshp commented Sep 24, 2022

Using AT

#134
#201

=> abaplint/transpiler#706

FOR loops with identical declared variable names

#194
#200
#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

#181

abaplint/transpiler#669

@larshp larshp pinned this issue Sep 24, 2022
@BaerbelW
Copy link

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
Copy link
Member Author

larshp commented Sep 25, 2022

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
Copy link

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

Cheers
Bärbel

@larshp
Copy link
Member Author

larshp commented Nov 19, 2022

VALUE in DATA: will be fixed with exercism/abap-test-runner#168 when approved by @exercism/maintainers-admin

@larshp
Copy link
Member Author

larshp commented Nov 22, 2022

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

@larshp
Copy link
Member Author

larshp commented Mar 7, 2023

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

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