Skip to content

Sp 4818 update varsome api client to use commitizen#26

Merged
kchatzikok merged 2 commits intomasterfrom
SP-4818-update-varsome-api-client-to-use-commitizen
Mar 26, 2024
Merged

Sp 4818 update varsome api client to use commitizen#26
kchatzikok merged 2 commits intomasterfrom
SP-4818-update-varsome-api-client-to-use-commitizen

Conversation

@thasteris
Copy link
Copy Markdown
Contributor

No description provided.

@n-zervos
Copy link
Copy Markdown

Great work @thasteris !

General question, wouldn't it be worth it to also add poetry to the pyproject.toml file so that poetry can handle dependency management? @kchatzikok Your thoughts?

@thasteris
Copy link
Copy Markdown
Contributor Author

Thanks @n-zervos , for your input, however in this phase in development we won't be implementing poetry, this will be moved to a later phase. @kchatzikok Do you agree?

Copy link
Copy Markdown

@bsantor bsantor left a comment

Choose a reason for hiding this comment

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

Great job @thasteris! 👍🏼

@kchatzikok
Copy link
Copy Markdown
Contributor

@n-zervos @thasteris it's ok like that for now. We might consider adding poetry for dependency management in the future

@kchatzikok kchatzikok merged commit bba69f8 into master Mar 26, 2024
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.

4 participants