clj-commons

danielcompton 2023-12-21T10:03:04.569509Z

@slipset are you able to take a look at the CI in https://github.com/clj-commons/hickory/commit/70d62f55d3c2a67869d092deb2bd23250d7521a1? The GitHub Actions bit is working, but CircleCI is failing

danielcompton 2023-12-21T10:03:33.100159Z

I’m happy to switch it all over to GitHub Actions if there is a deployment script for that.

slipset 2023-12-21T10:26:22.115559Z

I’m a bit busy right now, but I’ll try to have a look at it some time today. Please remind me tomorrow if I forget.

danielcompton 2023-12-21T19:10:41.289349Z

Think I’ve got it all fixed now, and I’ve pushed a 0.7.4 release (and deprecated 0.7.2 and 0.7.3)

danielcompton 2023-12-21T19:10:53.229939Z

Because of https://github.com/clj-commons/hickory/issues/84

danielcompton 2023-12-21T18:54:15.647929Z

I don’t understand why this deploy job ran in CircleCI - https://app.circleci.com/pipelines/github/clj-commons/hickory/79/workflows/05080a6f-9c85-4fc6-8712-c1382fdd7500/jobs/115. The https://github.com/clj-commons/hickory/blob/1ad9a1f7a40bea4e2528a06f87f9464f47b0a280/.circleci/config.yml#L15-L22 has a filter for tags. It’s been a while since I did Circle work, does anyone else have an idea?

danielcompton 2023-12-21T18:54:44.761019Z

https://circleci.com/docs/configuration-reference/#tags

lread 2023-12-21T19:13:33.265049Z

not sure @danielcompton as i mostly use github actions these days, but maybe you have to ignore branches? https://discuss.circleci.com/t/workflow-job-filter-to-only-tags-not-working/40628

danielcompton 2023-12-21T19:14:48.773909Z

Yeah, I’m guessing that is the reason why we have https://github.com/clj-commons/hickory/blob/1ad9a1f7a40bea4e2528a06f87f9464f47b0a280/.circleci/config.yml#L137-L143, @borkdude?

slipset 2023-12-21T19:17:57.659399Z

That’s me.

slipset 2023-12-21T19:18:37.846879Z

I think @tiagomlalves did something to sort this out in clj-ssh

slipset 2023-12-21T19:19:04.078049Z

Check the commits on the circle config there

Tiago Alves 2023-12-21T21:33:21.625479Z

@danielcompton the reason is that CircleCI needs a rule for branches

Tiago Alves 2023-12-21T21:34:44.216299Z

this was a bit weird to me coming from GitHub Actions, but it’s well documented in CircleCI

Tiago Alves 2023-12-21T21:35:16.874039Z

feel free to ping me if you need further assistance

👍 1