search

IntersectMBO / cardano-cli

This repository contains sources for the command-line interface (CLI) tool for interacting with the Cardano blockchain.
Apache License 2.0
37 stars 13 forks source link

release-upload.yaml: simplify logic by aggregating steps #760

Closed smelc closed 3 months ago

smelc commented 3 months ago

Changelog

- description: |
    Simplify logic of releasing pipeline
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  - refactoring    # QoL changes
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

Follow-up of earlier discussions that were left opened on https://github.com/IntersectMBO/cardano-cli/pull/749

How to trust this PR

I executed this pipeline manually and it goes through the modified steps nicely: https://github.com/IntersectMBO/cardano-cli/actions/runs/9068498972/job/24915930335

Checklist

smelc commented 3 months ago

Note to reviewers: I will fixup the Retrigger Hydra ¯ \ _ (ツ) _ / ¯ commit before merging, so that it does not show up in the history :wink:

smelc commented 3 months ago

I'll merge this one after https://github.com/IntersectMBO/cardano-cli/pull/761