Skip to content
This repository has been archived by the owner on Apr 26, 2020. It is now read-only.

fix: make sentry release work without git #228

Merged
merged 1 commit into from
Jul 14, 2019

Conversation

atymic
Copy link
Contributor

@atymic atymic commented Jul 14, 2019

Q A
Bug fix? Yes
New feature? No
BC breaks? No
Deprecations? No
Fixed tickets #227

I also added the lock file to the gitignore, as it's not included in packages.

Do not forget to add notes about your changes to [CHANGELOG.md]

Do I add a new version to add to the change log?

@@ -130,12 +135,17 @@ static function (&$value) use ($config) {
}
);

function releaseIsGitDirectory()
{
return (bool) run('cd {{release_path}} && git rev-parse --git-dir > /dev/null 2>&1 && echo 1');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we need to support deploying to windows servers?

},
$lines
);
array_map('trim', array_slice(explode("\n", $result), 0, 200)),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have reformatted this to PSR-2, can revert if you would like.

@antonmedv antonmedv merged commit 2fe3999 into deployphp:master Jul 14, 2019
endelwar added a commit to endelwar/recipes that referenced this pull request Sep 22, 2019
@endelwar endelwar mentioned this pull request Sep 22, 2019
antonmedv pushed a commit that referenced this pull request Sep 23, 2019
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.

None yet

2 participants