Skip to content

Update Reference Solution for Exercise "Book Store" (Replace Workaround)#216

Draft
bau-mann wants to merge 7 commits into
exercism:mainfrom
bau-mann:book_store_main
Draft

Update Reference Solution for Exercise "Book Store" (Replace Workaround)#216
bau-mann wants to merge 7 commits into
exercism:mainfrom
bau-mann:book_store_main

Conversation

@bau-mann

@bau-mann bau-mann commented Oct 6, 2022

Copy link
Copy Markdown
Contributor

resolves: #215

@bau-mann

bau-mann commented Oct 6, 2022

Copy link
Copy Markdown
Contributor Author

@larshp Could you take a look? The solution runs ob BTP ABAP Environment w/ all unit tests passing.

@larshp

larshp commented Oct 7, 2022

Copy link
Copy Markdown
Member

downport of FILTER is missing = abaplint/abaplint#2720

@bau-mann feel free to join the abapGit slack channel, https://abapgit-slackinviter.herokuapp.com (this is a public invite link, so everyone is welcome), we have a #exercism channel there

@larshp

larshp commented Oct 7, 2022

Copy link
Copy Markdown
Member

hmm, got some problems with the downport,

image

@bau-mann bau-mann changed the title Implement Exercise "Book Store" Update Reference Solution for Exercise "Book Store" (Replace Workaround) Oct 14, 2022
@mbtools

mbtools commented Oct 3, 2023

Copy link
Copy Markdown
Contributor

tried with latest and it works:

    "@abaplint/transpiler-cli": "^2.7.101",
    "@abaplint/cli": "^2.102.54",
    "@abaplint/runtime": "^2.7.101"

merge?

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

Successfully merging this pull request may close these issues.

Implement Exercise "Book Store"

3 participants