search

fidley / ABAPQuickFix

ABAP Quick Fix
https://abapblog.com
MIT License
42 stars 9 forks source link

Add extension point in order to be able to add additional fixes. #39

Closed lc-leuc closed 2 years ago

lc-leuc commented 2 years ago

Hi Łukasz,

please consider to make ABAPQuickFixes extensible, in order to provide specialized add-ons.

Best regards Lawrence

fidley commented 2 years ago

Hello Lawrence,

nice idea! I will check if there are no build errors and I'll publish the new version. Have you maybe prepared short instruction for extending the plugin to make it easier for others?

PS: I'm on a business trip so it can take few days until I publish the merged version. Cheers Łukasz

fidley commented 2 years ago

Hello Lawrence, I will try to build it today and publish the version on the update site.

Thanks! Łukasz

lc-leuc commented 2 years ago

Hi Łukasz,

great news. I'm looking forward to being able to publish the extension, soon.

Best regards Lawrence

fidley commented 2 years ago

Hi Lawrence,

new version is published :) Waiting for your addon now :)

Cheers Łukasz

lc-leuc commented 1 year ago 
Hi Łukasz,

great. Please take your time. An extension instruction is a nice idea.

However, I have exposed only necessary packages so far. This ought to be discussed.

In order to provide a general extensibility approach a careful choice of (stable) and useful API should be identified.  

So far only c.a.a.quickfix, c.a.a.q.a.s.codeParser, c.a.a.q.a.s.statements and c.a.a.q.a.s.statements.move are included in the export list.

After having a second look, maybe this is already a good start.

Have a nice trip.

Best regards Lawrence

Łukasz Pęgiel ***@***.***> hat am 11.05.2022 10:24 geschrieben:

Hello Lawrence, nice idea! I will check if there are no build errors and I'll publish the new version. Have you maybe prepared short instruction for extending the plugin to make it easier for others? PS: I'm on a business trip so it can take few days until I publish the merged version.CheersŁukasz —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>