Skip to content

Conversation

@mhaas
Copy link
Contributor

@mhaas mhaas commented Jul 3, 2020

No description provided.

@mhaas mhaas requested a review from karthikmswamy July 3, 2020 09:01
Copy link
Contributor

@karthikmswamy karthikmswamy left a comment

Choose a reason for hiding this comment

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

Quick question: would anyone creating a release need accounts/permissions to push to pypi?

@@ -0,0 +1,45 @@
Creating a new Release of the SDK
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a minor point: would this be more clear about which SDK - Creating a new release of the Data Attribute Recommendation Python SDK?

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, I do not feel release needs to be capitalised, but this is just my opinion.

@karthikmswamy
Copy link
Contributor

By the way, this is really well documented and very useful / important information. Thanks for adding this :)

@mhaas
Copy link
Contributor Author

mhaas commented Jul 3, 2020

Hi @karthikmswamy, thanks for the pointers. Does the latest update clarify your questions?

Copy link
Contributor

@karthikmswamy karthikmswamy left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for adding this!

@mhaas
Copy link
Contributor Author

mhaas commented Jul 3, 2020

Fixes #22

@mhaas mhaas merged commit 24c1add into master Jul 3, 2020
@mhaas mhaas deleted the feat/add_release_guide branch July 3, 2020 09:24
@mhaas mhaas mentioned this pull request Jul 3, 2020
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.

3 participants