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

Delete objects moved to abap2xlsx/demos repository #998

Merged

Conversation

sandraros
Copy link
Collaborator

@sandraros sandraros commented Mar 6, 2022

As discussed in #861, proposing to remove the demo programs from the main repository. I have just created the repository https://github.com/abap2xlsx/demos with all the demo programs. Hopefully it will also solve #993 (complex issue related to abapGit way of pulling and combination of DDIC, programs, classes and transformations).

Copy link
Member

@AndreaBorgia-Abo AndreaBorgia-Abo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
One question, though: the linter complaints are due to the move, right?

@sandraros
Copy link
Collaborator Author

I didn't see that. I have added abaplint.json and changed the rules a little bit, compared to abap2xlsx/abap2xlsx. Now it's fine.

@AndreaBorgia-Abo
Copy link
Member

@sandraros do we want to move the W3MI objects as well? This is not needed for the abapGit issue but the subpackage is not otherwise used.

@sandraros
Copy link
Collaborator Author

@AndreaBorgia-Abo Oh yes, sure, I forgot them!

@AndreaBorgia-Abo
Copy link
Member

AndreaBorgia-Abo commented Mar 26, 2022

Moving season.... PRs incoming....
@sandraros I can't assign the PR to you under "demos", weird.
The other one was fine.

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.

None yet

2 participants