search

cloudsmith-io / action

Github Action that uses the cloudsmith cli to interact with the Cloudsmith API (pushes, etc)
https://github.com/marketplace/actions/cloudsmith-push
MIT License
14 stars 12 forks source link

Add support for setting tags after push #27

Closed jtojnar closed 2 years ago

jtojnar commented 2 years ago

Passing --tags in extra argument does not work for version:latest for some reason so we need to do that in a separate step.

Fixes: https://github.com/cloudsmith-io/action/issues/16

jtojnar commented 2 years ago

Seems to work now: https://github.com/fossar/selfoss/runs/6410387465?check_suite_focus=true#step:10:108

dylwylie commented 2 years ago

Hey @jtojnar, afraid this seems to result in some issues

running

test/libs/bats/bin/bats test/entrypoint.bats

results in push tests failing due to a non successful exit code

and the push-raw workflow results in

Getting list of packages ... OK Traceback (most recent call last): File "<string>", line 6, in <module> AssertionError: Query “filename:test/fixture/raw_file.txt version:4900c03962d6a8d37b700a20718c1dee87fba48e” needs to match a single package in repository “cloudsmith/actions” to be able to add tags. Usage: cloudsmith tags add [OPTIONS] OWNER/REPO/PACKAGE TAGS Try 'cloudsmith tags add -h' for help. Error: Invalid value for 'OWNER/REPO/PACKAGE': Individual values cannot be blank

jtojnar commented 2 years ago

Sorry, I was not able to run the tests locally and CI did not run on GitHub because it needs to be triggered manually for people that did not contribute to the repository in the past. Will try to give it another look in #30 but I am not really sure why the test fails – it seems to work in practice.