Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add release name to deploy:info #3783

Merged
merged 2 commits into from
Feb 26, 2024
Merged

Add release name to deploy:info #3783

merged 2 commits into from
Feb 26, 2024

Conversation

Propaganistas
Copy link
Contributor

I always find myself adding a task to just show the release name.
Why not add it to the built-in deploy:info task?

@Schrank
Copy link
Contributor

Schrank commented Feb 26, 2024

Nice idea! But stupid question: where is the release_name taken from? 😅

@Propaganistas
Copy link
Contributor Author

Propaganistas commented Feb 26, 2024

@Schrank

set('release_name', function () {
return within('{{deploy_path}}', function () {
$latest = run('cat .dep/latest_release || echo 0');
return strval(intval($latest) + 1);
});
});

It just reads the (potentially) already existing releases log.
And because this file is always loaded, the definition of release_name will always be available.

@Propaganistas
Copy link
Contributor Author

Added a failsafe for initial deployments

@antonmedv antonmedv merged commit ef0087d into deployphp:master Feb 26, 2024
9 checks passed
@Propaganistas Propaganistas deleted the Propaganistas-patch-1 branch February 26, 2024 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants