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

refactor: create a ContextMixin class, and extract methods into there #494

Merged
merged 12 commits into from
Feb 18, 2022
Merged

refactor: create a ContextMixin class, and extract methods into there #494

merged 12 commits into from
Feb 18, 2022

Conversation

Catalyst4222
Copy link
Contributor

About

This pull request adds in a ContextMixin class, inheriting from DictSerializerMixin and designed to be used in the creation of new context classes

Checklist

  • I've ran pre-commit to format and lint the change(s) made.
  • I've checked to make sure the change(s) work on 3.8.6 and higher.
  • This fixes/solves an Issue.
    • (If existent): Adds in one mixin from 487
  • I've made this pull request for/as: (check all that apply)
    • Documentation
    • Breaking change
    • New feature/enhancement
    • Bugfix

interactions/context.py Outdated Show resolved Hide resolved
interactions/context.py Show resolved Hide resolved
interactions/context.py Show resolved Hide resolved
interactions/context.py Outdated Show resolved Hide resolved
interactions/context.py Outdated Show resolved Hide resolved
interactions/context.py Outdated Show resolved Hide resolved
interactions/context.pyi Outdated Show resolved Hide resolved
interactions/context.pyi Outdated Show resolved Hide resolved
interactions/context.pyi Outdated Show resolved Hide resolved
@i0bs i0bs linked an issue Feb 18, 2022 that may be closed by this pull request
1 task
@i0bs i0bs enabled auto-merge (squash) February 18, 2022 19:50
Copy link
Contributor

@EdVraz EdVraz left a comment

Choose a reason for hiding this comment

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

I read it like 10 times, I hope the code is fine

@i0bs i0bs merged commit e0161c6 into interactions-py:unstable Feb 18, 2022
i0bs pushed a commit that referenced this pull request Feb 25, 2022
…#494)

* refactor: create a ContextMixin class, and extract methods into there

* docs: update context typehints and pyi file

* ci: correct from checks.

* fix: adjust payload creation

* ci: correct from checks.

* ci: correct from checks.

* refactor: change ContextMixin to _Context

* fix: forgotten name change

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.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.

[REQUEST] Mixin implementations
3 participants