Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[semver:minor] Allow CIRCLE_TAG to be used for deployments, fail if no tag or branch. #8

Merged
merged 2 commits into from
Jul 31, 2020

Conversation

campbecf
Copy link
Contributor

#7

Should address this issue, as well as failing whenever a branch or tag cannot be found. This seems safer than just running on potentially an empty string.

I'm generally unfamiliar with bash scripting and conditionals, so I would appreciate if anyone else can double check my work / syntax.

@gmemstr
Copy link
Contributor

gmemstr commented Jul 16, 2020

@campbecf Looks like the alpha version of the orb expired, mind pushing a small/empty commit so we can re-trigger the tests?

Copy link
Contributor

@gmemstr gmemstr left a comment

Choose a reason for hiding this comment

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

Looks good, just a minor typo to fix.

src/commands/deploy-via-git.yml Outdated Show resolved Hide resolved
@gmemstr gmemstr changed the title Allow CIRCLE_TAG to be used for deployments, fail if no tag or branch. [semver:minor] Allow CIRCLE_TAG to be used for deployments, fail if no tag or branch. Jul 31, 2020
@gmemstr gmemstr merged commit 99d98f2 into CircleCI-Public:master Jul 31, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants