Skip to content

add certifi dep#12

Merged
biglovisa merged 2 commits into
mainfrom
ta/certifi
Feb 9, 2021
Merged

add certifi dep#12
biglovisa merged 2 commits into
mainfrom
ta/certifi

Conversation

@thdaraujo

@thdaraujo thdaraujo commented Feb 9, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

What

  • missing certifi dependency

SDK Release Checklist

  • Have you added an integration test for the changes?
  • Have you built the library locally and made queries against it successfully?
  • Did you update the changelog?
  • Did you bump the package version?
  • For breaking changes, did you plan for the release of the new SDK versions and deploy the API to production?

@thdaraujo thdaraujo self-assigned this Feb 9, 2021
@thdaraujo
thdaraujo requested a review from biglovisa February 9, 2021 18:36
biglovisa
biglovisa approved these changes Feb 9, 2021
View reviewed changes

@biglovisa biglovisa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🙌🏻

@biglovisa
biglovisa merged commit a68e492 into main Feb 9, 2021
@biglovisa
biglovisa deleted the ta/certifi branch February 9, 2021 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@biglovisa biglovisa biglovisa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@thdaraujo thdaraujo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@thdaraujo @biglovisa