Marc Bernard

Took me a while... empty cell = repeated value from above... typical ALV  Clear boundaries between layers certainly improves longevity 💯 for it

@sbcgua can you do this "study" again with current code and the settings I mentioned? or put the tool into a gist, please?

nice tool, thx jumper is fine. we would have to take it out of `zif_abapgit_object` which would move object-specific logic into several places. the `http` password dialog is the bad...

suggestion to add the following methods (and keep `de/serialize`): - `before_serialize` (similar to `before_export` in transports) - `after_serialize` - `before_deserialize` - `after_deserialize` (similar to `after_import` in transports) pseudo logic (same...

Example deserialize: Longterm, it lets us split then process into reading and validating all files first (before deserialize), then import all of them (deserialize), followed by the after import method...

Unfortunately, there's a conflict between experimental features. Meaning you can't test LXE and JSON for INTF separately. Remove the following lines: https://github.com/abapGit/abapGit/blob/6369c57c5646e1e12b39aa69802c48f82cafee72/src/objects/aff/zcl_abapgit_aff_registry.clas.abap#L58-L59

#6393 will allow enabling features independent of each other. Note, LXE is a repo setting and *not* related to the experimental feature flag (which is a global setting). 

Just keep in mind that so far LXE is enabled for a subset of object types: https://github.com/abapGit/abapGit/blob/9f8862809068ac44ee47c7a5a8170a00517bcd04/src/objects/texts/zcl_abapgit_lxe_texts.clas.abap#L202-L216

Re: end-of-file... AG always has an empty newline (0a) at the end of source code (in Diff, turn on View > Show Hidden Characters). The repo does not and needs...

Low hanging won't get us far. I'm all for refactoring, but something major like this should go into an AG 2.0 branch.