Skip to content

PLT-1207 ci: add Publishing workflow#8

Merged
KhueDuong merged 1 commit into
masterfrom
plt-1207/add-ci-pipeline-to-publish-to-s3pypi
Oct 7, 2025
Merged

PLT-1207 ci: add Publishing workflow#8
KhueDuong merged 1 commit into
masterfrom
plt-1207/add-ci-pipeline-to-publish-to-s3pypi

Conversation

@KhueDuong

@KhueDuong KhueDuong commented Oct 6, 2025

Copy link
Copy Markdown

This PR adds:

  • A workflow to build and publish the package to s3pypi
  • A ci workflow which runs mypy, ruff, test
  • pinning python dependencies

@KhueDuong KhueDuong force-pushed the plt-1207/add-ci-pipeline-to-publish-to-s3pypi branch 2 times, most recently from 5fe22bf to 2a2f383 Compare October 6, 2025 05:45
Comment thread .github/workflows/publish.yaml Outdated
Comment thread .github/workflows/publish.yaml Outdated
Comment thread pyproject.toml Outdated
@KhueDuong KhueDuong force-pushed the plt-1207/add-ci-pipeline-to-publish-to-s3pypi branch 12 times, most recently from 2480b1c to 43be7d7 Compare October 7, 2025 01:40
@KhueDuong

Copy link
Copy Markdown
Author

@KhueDuong KhueDuong force-pushed the plt-1207/add-ci-pipeline-to-publish-to-s3pypi branch from 43be7d7 to 6019732 Compare October 7, 2025 01:41
@KhueDuong

Copy link
Copy Markdown
Author

We have too many type errors, I can try to fix them all in another PR

Comment thread .github/workflows/publish.yaml Outdated
issues: write
jobs:
Publish:
uses: uptick/actions/.github/workflows/ci.yaml@plt-1207/add-publish-to-private-pypi-step

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I'll change this to uptick/actions/.github/workflows/ci.yaml@main before merging

@KhueDuong KhueDuong force-pushed the plt-1207/add-ci-pipeline-to-publish-to-s3pypi branch from 6019732 to 5b0f6f4 Compare October 7, 2025 02:34
@KhueDuong KhueDuong merged commit 43ae5ef into master Oct 7, 2025
1 check failed
@KhueDuong KhueDuong deleted the plt-1207/add-ci-pipeline-to-publish-to-s3pypi branch October 7, 2025 02:39
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