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

ci(gha): set up CI #4

Merged
merged 16 commits into from
Dec 20, 2023
Merged

ci(gha): set up CI #4

merged 16 commits into from
Dec 20, 2023

Conversation

isac322
Copy link
Member

@isac322 isac322 commented Dec 20, 2023

  • Configure mypy
  • Fix some type errors
  • Configure CI/CD using Github Actions
  • Minor update for README

@thefron I think we should drop Python 3.8. The recent versions of pydantic.BaseModel and Pydantic's dataclass are both not fully compatible with Python 3.8 (related issue). Probably I can handle it using some tweaks, but I do not think it is worth it.

Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
@isac322 isac322 requested a review from thefron December 20, 2023 02:43
@isac322 isac322 self-assigned this Dec 20, 2023
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Signed-off-by: Isac Byeonghoon Yoo <[email protected]>
Copy link
Member

@thefron thefron left a comment

Choose a reason for hiding this comment

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

LGTM! Also agree on dropping support up to Python 3.8.

run: |
pip install -U build
python -m build .
- name: Publish package distributions to PyPI
Copy link
Member

Choose a reason for hiding this comment

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

How are we bumping the package version while publishing? Can the version be automatically reflected to sdk version header?

Copy link
Member Author

Choose a reason for hiding this comment

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

How are we bumping the package version while publishing?

Just update variable __version__ of plugbear/__init__.py. Flit will pick the variable and publish with it.

Can the version be automatically reflected to sdk version header?

Yeap. Because the __version__ is an ordinary python variable, and we use it on header directly.

@isac322 isac322 merged commit 80e9756 into master Dec 20, 2023
4 checks passed
@isac322 isac322 deleted the setup-gha branch December 20, 2023 04:08
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