-
Notifications
You must be signed in to change notification settings - Fork 4
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(pdm): add pdm
actions (VG-14148)
#115
Conversation
8b90303
to
4959498
Compare
9d10146
to
2134415
Compare
pdm/release/action.yml
Outdated
# Disabled until Nexus is confirmed working | ||
# - name: Push to our internal Nexus | ||
# if: inputs.kind == 'lib' && env.PDM_PUBLISH_USERNAME != null && env.PDM_PUBLISH_PASSWORD != null | ||
# env: | ||
# PDM_PUBLISH_REPO: https://{% raw %}${{ secrets.GREEN_NEXUS_HOST }}{% endraw %}/repository/pypi-internal/ | ||
# PDM_PUBLISH_USERNAME: {% raw %}${{ secrets.GREEN_NEXUS_USER }}{% endraw %} | ||
# PDM_PUBLISH_PASSWORD: {% raw %}${{ secrets.GREEN_NEXUS_PASSWORD }}{% endraw %} | ||
# run: pdm publish --no-build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's working and used by at least a project so we should keep it enabled for publication of libraries.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad to hear it because I have been asking for the past year and each time I was told "it's not working" and then "it's not the target anymore, we are focusing on artifactory pro".
I'll be glad to enable it 🎉 (but I'll need access to test it's working)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well I know that at least on project uses it so it better fills it.
However we might drop it to artifactory in the near future for multiple reasons but the discussions are ongoing.
In the meanwhile we want to keep this activated :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I uncommented it, fixed it (but I haven't tested it yet)
ac1f47c
to
886cd4b
Compare
6cfd537
to
87ef125
Compare
a1ea5f7
to
13ecd0a
Compare
13ecd0a
to
a54dd1f
Compare
Provides
pdm
actions for the entire lifecycle of apdm
-based project, lib or service.Sample runs
goss
: https://github.com/LedgerHQ/les-copier-demo/actions/runs/7802772748VG-14148