feat: remove SQL repository

[joaoandre-avaiga]

No description provided.

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
18934	16034	85%	0%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
taipy/core/_entity/_migrate/init.py	100%	🟢
taipy/core/_entity/_migrate_cli.py	94%	🟢
taipy/core/_repository/_base_taipy_model.py	93%	🟢
taipy/core/_version/_version_manager_factory.py	89%	🟢
taipy/core/_version/_version_model.py	93%	🟢
taipy/core/cycle/_cycle_manager_factory.py	89%	🟢
taipy/core/cycle/_cycle_model.py	95%	🟢
taipy/core/data/_data_manager_factory.py	89%	🟢
taipy/core/data/_data_model.py	96%	🟢
taipy/core/job/_job_manager_factory.py	89%	🟢
taipy/core/job/_job_model.py	95%	🟢
taipy/core/scenario/_scenario_manager_factory.py	89%	🟢
taipy/core/scenario/_scenario_model.py	96%	🟢
taipy/core/submission/_submission_manager_factory.py	89%	🟢
taipy/core/submission/_submission_model.py	96%	🟢
taipy/core/task/_task_manager_factory.py	89%	🟢
taipy/core/task/_task_model.py	95%	🟢
TOTAL	93%	🟢

updated for commit: 60adf5d by action🐍

[toan-quach]

lgtm! a friendly note that some of the changes should be applied to enterprise repo as well :D

[toan-quach]

I forgot but do check taipy-config as well to see if there's any removals to be done

[jrobinAV]

The code coverage of taipy/core/_repository/_base_taipy_model.py is slightly low. Can you check if we can improve the coverage of this class?

[joaoandre-avaiga]

I forgot but do check taipy-config as well to see if there's any removals to be done

I've checked config, theres no direct reference to sql repo, only the repository attributes(type, properties), we should be good. I'll check enterprise if theres anything to be done.

[joaoandre-avaiga]

The code coverage of taipy/core/_repository/_base_taipy_model.py is slightly low. Can you check if we can improve the coverage of this class?

I've added specif tests for the base_model class. There are 2 abstract methods still not covered, but they don't have any logic to be tested. So this should be enough.

[trgiangdo]

This is looking good. Thank you Joao

[jrobinAV]

As people were saying in the 50's:
"Rock-and-roll"...