Skip to content

[dev]: run macos-amd64 job on release PRs #2970

Closed

Description

@jsjoeio

Due to the weirdness of our release process, we overlooked the changes to our CI workflow when we merged #2866.

We turned off CI for macos-amd64 and only run it on the default branch.

Unfortunately the release process relies on the artifacts created during a release PR (example: #2953) when uploading release assets. This is a limitation in our workflow that we need to address.

Current workaround: build macos-amd64 locally and manually upload to release assets.

Notes

Ideally, we look into a better process for the release assets. The current solution isn't great.

Release off a tagged commit instead.

Activity

  1. added
    bugSomething isn't working
    ciIssues related to ci
    on Mar 25, 2021
  2. added this to the v3.9.3 milestone on Mar 25, 2021
  3. code-asher commented on Mar 26, 2021

    @code-asher
    Member

    Yeah I think maybe instead of releasing off a PR we could release off the tagged commit in main.

  4. jsjoeio commented on Mar 26, 2021

    @jsjoeio
    ContributorAuthor

    Good call! That would save a lot of trouble.

  5. modified the milestones: v3.9.3, On Deck, v3.9.4 on Apr 8, 2021
  6. self-assigned this
    on Apr 15, 2021
  7. oxy commented on Apr 15, 2021

    @oxy

    We actually resolved this in the major nuke-CI-and-rewrite it; I just forgot to close the issue 😅

  8. added
    choreRelated to maintenance or clean up
    on May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingchoreRelated to maintenance or clean upciIssues related to ci

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions