Skip to content

ilicompiler integration#160

Open
signedav wants to merge 2 commits into
mainfrom
ili2cleanlib
Open

ilicompiler integration#160
signedav wants to merge 2 commits into
mainfrom
ili2cleanlib

Conversation

@signedav

@signedav signedav commented Nov 11, 2025

Copy link
Copy Markdown
Member

ilicompiler integration

Integrate IliCompiler functionality in the iliwrapper module.

Some restructuring on the ili2db functions where it made sense.

Keeping backwards compatibility for:

  • Modules ili2dbutils and ili2dbconfig and ili2dbutils keep the db in the name (althoug they also contain ili2c stuff, because they use common functions classes)

Breaking backwards compatibility for:

  • IliExecutable has now abstract functions, but should not be a problem, because it's usually derived (by IliImporter/IliExporter etc.) for the library usage

@signedav signedav changed the title PoC ili2py R&D Backend - Don't merge! ilicompiler integration Jul 3, 2026
@signedav signedav marked this pull request as ready for review July 3, 2026 11:14
@signedav signedav requested a review from gacarrillor July 3, 2026 11:55

@gacarrillor gacarrillor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice cleanups and new docs. 👍🏽

Comment thread modelbaker/iliwrapper/ili2dbutils.py Outdated
Comment thread modelbaker/iliwrapper/ili2dbutils.py Outdated
@signedav signedav force-pushed the ili2cleanlib branch 3 times, most recently from 9c42bce to fa8a46b Compare July 9, 2026 07:14
signedav added 2 commits July 10, 2026 17:15
Some restructuring on the ili2db functions but keeping backwards compatibility (except the iliExcecutable, but should not be a problem, because it's usually derived (by IliImporter/IliExporter etc.) for the library usage
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.

2 participants