Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Publish Node.js binaries on tag pushes in Bitrise #6789

Merged
merged 1 commit into from
Oct 22, 2016

Conversation

mikemorris
Copy link
Contributor

@mikemorris mikemorris commented Oct 21, 2016

Fixes #4854

Refs #6523, #6525

Bitrise recently added the ability to trigger workflows from git tag pushes, which finally unblocked this!

I can't seem to get this configured correctly following the instructions in https://bitrise-io.github.io/devcenter/webhooks/trigger-map/ - tag pushes to an updated webhook URL are being rejected by Bitrise.

{"success_responses":[],"failed_responses":[{"status":"error","message":"trigger pattern did not match any defined mapping: no matching workflow found with trigger params: push-branch: , pr-source-branch: , pr-target-branch: , tag: node-v3.4.0-pre.11","service":"bitrise","slug":"55e3a9bf71202106","build_slug":"","build_number":0,"build_url":"","triggered_workflow":""}]}

Is there anything special I need to do to get Bitrise to recognize the tag: "*" workflow from platform/node/bitrise.yml?

(Side note: Bitrise env vars handling is done using https://github.com/bitrise-io/envman/)

/cc @1ec5 @incanus

@mikemorris
Copy link
Contributor Author

I needed to update bitrise.yml in the Bitrise UI Workflow Editor to forward tag triggers on to platform/node/bitrise.yml

Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it works, ship it!

@mikemorris
Copy link
Contributor Author

@1ec5 not publishing binaries yet, but it's at least entering the Bitrise workflow for tag pushes now 😄

@mikemorris
Copy link
Contributor Author

mikemorris commented Oct 22, 2016

Pushing node-v* tags now publish macOS binaries again!!

@mikemorris mikemorris merged commit eb1b385 into master Oct 22, 2016
@mikemorris mikemorris deleted the 4854-node-bitrise-publish branch October 22, 2016 04:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants