Skip to content

Updated the README#222

Merged
lalitpagaria merged 5 commits intoobsei:masterfrom
0xku:updated-docs
Jan 21, 2022
Merged

Updated the README#222
lalitpagaria merged 5 commits intoobsei:masterfrom
0xku:updated-docs

Conversation

@0xku
Copy link
Copy Markdown
Contributor

@0xku 0xku commented Jan 21, 2022

  • Updated the README to fix some typos and grammatical errors
  • Fixed a broken import
  • Quite a few pre-commit changes made by black

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 21, 2022

CLA assistant check
All committers have signed the CLA.

@lalitpagaria
Copy link
Copy Markdown
Collaborator

@kuutsav Thank you for working on it.
I will review and share feedback with you.

@lalitpagaria
Copy link
Copy Markdown
Collaborator

Kindly create separate PR for black formatting-related changes, as it would be easier to review.

This reverts commit 5fbf7a8.
@0xku 0xku changed the title Updated the README and fixed a broken import Updated the README Jan 21, 2022
@0xku
Copy link
Copy Markdown
Contributor Author

0xku commented Jan 21, 2022

Kindly create separate PR for black formatting-related changes, as it would be easier to review.

Reverted two commits. This PR just contains the updates to the README now.

Copy link
Copy Markdown
Collaborator

@lalitpagaria lalitpagaria left a comment

Choose a reason for hiding this comment

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

Thanks, @kuutsav for fixing Readme.
I have one comment, the rest all is fine.

README.md Outdated
Comment on lines +801 to +806
# For custom domain refer http://docs.facetoe.com.au/zenpy.html#custom-domains
# Mainly you can do this by setting the environment variables:
# ZENPY_FORCE_NETLOC
# ZENPY_FORCE_SCHEME (default to https)
# when set it will force request on:
# {scheme}://{netloc}/endpoint
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These lines are stale now, I forgot to remove it in #221

@0xku
Copy link
Copy Markdown
Contributor Author

0xku commented Jan 21, 2022

Thanks, @kuutsav for fixing Readme. I have one comment, the rest all is fine.

Done!

@lalitpagaria lalitpagaria merged commit 79ab871 into obsei:master Jan 21, 2022
@0xku 0xku deleted the updated-docs branch January 21, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants