Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(semantic-release): add semantic release to the project #114

Merged
merged 8 commits into from
Feb 3, 2023

Conversation

luabida
Copy link
Collaborator

@luabida luabida commented Feb 3, 2023

I've also noticed requirements.txt and a setup.py file, I chose to use the poetry to do the build, since setuptools is not as dynamic

@luabida luabida requested a review from xmnlab February 3, 2023 14:28
@luabida
Copy link
Collaborator Author

luabida commented Feb 3, 2023

I'm not sure if PYPI_TOKEN is present in the secrets, I don't have access to check it out

@xmnlab
Copy link
Member

xmnlab commented Feb 3, 2023

@fccoelho is the only maintainer there: https://pypi.org/project/pysus/
@fccoelho could you generate a token there and create a secret here called PYPI_TOKEN with that token pls?

@xmnlab
Copy link
Member

xmnlab commented Feb 3, 2023

in general looks good to me.
just one thing, could you add to the release workflow the options for PR and push?

you can take a look at here https://github.com/arxlang/arx/blob/main/.github/workflows/release.yaml#L6

check that some steps has if statements specific for workflow or for "not" workflow

@@ -99,10 +98,6 @@ def test_type_convertion(self):
self.assertTrue(self.dbf_name.exists())
assert not all(df.dtypes == "object")

def test_geocode(self):
self.assertTrue(self.data_pickle.exists())
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I didn't see much use for this test..

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't remember if at some point sinan generates this pickle file, but from the name of the test it seems that this test is incomplete.

def test_read_dbc(self):
df = read_dbc(str(self.dbc_fname))
self.assertIsInstance(df, pd.DataFrame)
self.assertGreater(df.size, 0)

def test_read_dbc_dbf(self):
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've split this test and remove the duplicated part, that is now skipped

Copy link
Collaborator

@esloch esloch left a comment

Choose a reason for hiding this comment

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

LGTM!

@esloch esloch merged commit b079089 into AlertaDengue:master Feb 3, 2023
@github-actions
Copy link

github-actions bot commented Feb 6, 2023

🎉 This PR is included in version 0.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants