Skip to content
This repository has been archived by the owner on Jan 2, 2024. It is now read-only.

feature/#806 move name attribute of dn to properties #812

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

toan-quach
Copy link
Member

@toan-quach toan-quach force-pushed the feature/move-name-to-properties-dict-for-dn branch from d388f6e to 74a7ddb Compare November 2, 2023 09:38
Copy link

github-actions bot commented Nov 3, 2023

☂️ Python Cov

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8164 7755 95% 85% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/taipy/core/_entity/_migrate/_utils.py 85% 🟢
src/taipy/core/data/_abstract_sql.py 90% 🟢
src/taipy/core/data/_data_converter.py 97% 🟢
src/taipy/core/data/_data_manager.py 99% 🟢
src/taipy/core/data/_data_model.py 100% 🟢
src/taipy/core/data/csv.py 95% 🟢
src/taipy/core/data/data_node.py 98% 🟢
src/taipy/core/data/excel.py 97% 🟢
src/taipy/core/data/generic.py 98% 🟢
src/taipy/core/data/in_memory.py 100% 🟢
src/taipy/core/data/json.py 95% 🟢
src/taipy/core/data/mongo.py 99% 🟢
src/taipy/core/data/parquet.py 98% 🟢
src/taipy/core/data/pickle.py 100% 🟢
src/taipy/core/data/sql.py 97% 🟢
src/taipy/core/data/sql_table.py 92% 🟢
TOTAL 96% 🟢

updated for commit: d51b336 by action🐍

trgiangdo
trgiangdo previously approved these changes Nov 3, 2023
Copy link
Member

@trgiangdo trgiangdo left a comment

Choose a reason for hiding this comment

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

LGTM

@trgiangdo trgiangdo requested a review from jrobinAV November 3, 2023 11:06
@trgiangdo trgiangdo force-pushed the feature/move-name-to-properties-dict-for-dn branch from a9efcc5 to d51b336 Compare November 3, 2023 11:10
@toan-quach toan-quach merged commit 63fcfb7 into develop Nov 6, 2023
41 of 42 checks passed
@toan-quach toan-quach deleted the feature/move-name-to-properties-dict-for-dn branch November 6, 2023 10:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants