Skip to content

Conversation

@KNanda552
Copy link
Contributor

Adding GitHub Actions workflow config file to Move-DAR-SDK-from Travis CI to Github actions.

Checks:

  • README.md is maintained
  • Documentation is maintained
  • CHANGELOG.md is updated
  • Tests added

@KNanda552 KNanda552 marked this pull request as ready for review February 20, 2025 10:34
Copy link

@Shegox Shegox left a comment

Choose a reason for hiding this comment

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

I have a few things which could be improved and a few nit picks, which you might want to take a look at.

Copy link

@Shegox Shegox left a comment

Choose a reason for hiding this comment

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

Also one more thing, we should directly remove the .travis.yml with this PR and adjust the required checks in your GitHub branch protection rule.

Copy link

@Shegox Shegox left a comment

Choose a reason for hiding this comment

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

Looks good 🚀

run: pipx run build --sdist --wheel

- name: Deploy release to PyPI
uses: pypa/gh-action-pypi-publish@release/v1 # Uses OIDC/Workload identities and doesn't require a token No newline at end of file
Copy link

Choose a reason for hiding this comment

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

Just to be sure, did you make sure to add the trusted publisher configuration in Pypi?
(https://docs.pypi.org/trusted-publishers/adding-a-publisher/)

@KNanda552 KNanda552 requested a review from eksutter February 24, 2025 10:11
Copy link
Contributor

@eksutter eksutter left a comment

Choose a reason for hiding this comment

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

Thanks! I have only a minor question to the used patch versions of Python. I would perhaps drop the patch version.

@KNanda552 KNanda552 requested a review from eksutter February 25, 2025 04:26
Copy link
Contributor

@eksutter eksutter left a comment

Choose a reason for hiding this comment

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

Thanks Krishna!

@KNanda552 KNanda552 merged commit 5848207 into main Feb 25, 2025
11 checks passed
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.

4 participants