dribia / driviz

Data Visualization Library containing the Dribia Theme
MIT License
0 stars 0 forks source link

3 minimal initial version #4

Closed hribera closed 1 month ago

codecov[bot] commented 1 month ago

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

hribera commented 1 month ago

@airibarne Crec que això seria la minimal initial version on els codis dels colors estan per allà al mig i el theme és tal qual ara copy&paste del template de projecte ml.

He fet una mica de documentació, però es pot millorar.

He afegit testos també al mòdul de la CLI.

Algunes d'aquestes coses m'ha fet trobar alguns bugs o veure coses que faltaven al dritemplate-library així que també els he anat incorporant allà.

airibarne commented 1 month ago

Vale genial, era la idea això d'anar trobant buguillos al template de llibreria :smile:

Ara que dius això de la CLI... Clar, aquesta no té sentit que tingui CLI, perquè sempre s'utilitzarà dins el codi.

Llavors:

hribera commented 1 month ago

Esborrat el mòdul de la CLI del driviz :fire:

hribera commented 1 month ago

@airibarne Crec que això ja està així com a minimal version.

mauro-dribia commented 1 month ago

tendría sentido mover el código a __init__ para poder hacer from driviz import theme en lugar de from driviz.theme import theme? @airibarne

airibarne commented 1 month ago

Agree, @hribera posa al __init__ un from .theme import theme i afegim el theme al __all__

hribera commented 1 month ago

@mauro-dribia @airibarne Fet!