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: add type information for all cloudevent member functions #173

Merged

Conversation

sasha-tkachev
Copy link
Contributor

@sasha-tkachev sasha-tkachev commented Jul 11, 2022

Changes

Add type information for all CloudEvent member functions

  • Tests pass
  • Appropriate changes to README are included in PR

Signed-off-by: Alexander Tkachev <sasha64sasha@gmail.com>
@xSAVIKx
Copy link
Member

xSAVIKx commented Jul 11, 2022

@sasha-tkachev LGTM. A couple of things though.

  1. You can use [x] instead of [v ] in the GH markdown in order to render a pretty checked checkbox for you, like this:
  • <- checked checkbox
  1. Can you please also mention your changes in the unreleased section of the CHANGELOG.md?

@sasha-tkachev
Copy link
Contributor Author

@xSAVIKx I have completed all of your requests :)

@xSAVIKx xSAVIKx merged commit f39b964 into cloudevents:master Jul 12, 2022
sasha-tkachev added a commit to sasha-tkachev/sdk-python that referenced this pull request Jul 17, 2022
…events#173)

* feat: add type information for all cloudevent member functions

Signed-off-by: Alexander Tkachev <sasha64sasha@gmail.com>

* docs: update changelog

Signed-off-by: Alexander Tkachev <sasha64sasha@gmail.com>
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.

2 participants