Skip to content

removed old parameter to fix syntax error #76

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

Merged
merged 1 commit into from
Apr 23, 2023

Conversation

marcellourbani
Copy link
Contributor

didn't upgrade my agbapgit in a while, when I did this broke for a small syntax error.
Once fixed works well enough to create this branch and commit with the plugin.

I see this is now deprecated. Is the plan to move to odense_z?
Or just lack of time to maintain? In th elatter case I guess I can pick this up

@larshp
Copy link
Member

larshp commented Apr 23, 2023

well, the backend logic must be in sync with the plugin

and the backend logic has not been updated by SAP in a long time, SAP must be the one publishing the SAP code

@marcellourbani
Copy link
Contributor Author

Ok, but what do you propose to support abapgit over ecclipse if this is obsolete?

@larshp
Copy link
Member

larshp commented Apr 23, 2023

the ADT plugin as such is not obsolete, it just has to be used with a SAP Steampunk backend

for now, use SAP GUI abapgit for on-prem

@larshp
Copy link
Member

larshp commented Apr 23, 2023

but anyhow, no problem merging this PR

@larshp larshp merged commit 89067be into abapGit:main Apr 23, 2023
@marcellourbani
Copy link
Contributor Author

lol, was about to do it myself :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants