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

feat: improve deployment config name parameter #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

batistadasilva
Copy link

Change the parameter type in order to support other compute platforms than EC2.

See https://docs.aws.amazon.com/codedeploy/latest/userguide/deployment-configurations.html

Make the parameter not required, so the valued configured in the deployment group is used.

See https://docs.aws.amazon.com/cli/latest/reference/deploy/create-deployment.html

Change the parameter type in order to support other compute platforms
than EC2.

See https://docs.aws.amazon.com/codedeploy/latest/userguide/deployment-configurations.html

Make the parameter not required, so the valued configured in the
deployment group is used.

See https://docs.aws.amazon.com/cli/latest/reference/deploy/create-deployment.html
@batistadasilva batistadasilva marked this pull request as ready for review November 17, 2023 08:39
@batistadasilva batistadasilva requested a review from a team as a code owner November 17, 2023 08:39
@batistadasilva batistadasilva marked this pull request as draft November 17, 2023 08:40
@batistadasilva batistadasilva marked this pull request as ready for review November 17, 2023 08:42
@batistadasilva
Copy link
Author

batistadasilva commented Nov 17, 2023

I think I won't be able to make the pipeline to pass on my fork, because it seems like it requires to have access to publish on circleci Orbs repository. What do you think I should do?

@batistadasilva
Copy link
Author

Do you have any news on this pull request?
Thank you

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.

1 participant