-
Notifications
You must be signed in to change notification settings - Fork 0
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 CI/CD Pipeline Stack Base Class #13
Conversation
# TEMPORARY -- we need to use this going forward | ||
# source = pipelines.CodePipelineSource.connection( | ||
# repo_string=github_config.repository, | ||
# branch=github_config.branch, | ||
# connection_arn=build_arn( | ||
# service="codestar-connections", | ||
# resource_type="connection", | ||
# resource_delim="/", | ||
# resource_id=github_config.codestar_connection, | ||
# ), | ||
# code_build_clone_output=True, | ||
# trigger_on_push=True, | ||
# ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might be an opportunity to switch to codestar
connection instead of legacy github connection? Or would you want to tackle this in separate work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made either option usable based on the presence of the fields in the source config.
BaseProjectConfig, | ||
CodePipelineSourceConfig, | ||
Env, | ||
GlobalConfig, | ||
PipelineConfig, | ||
ProjectConfig, | ||
StageConfig, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be useful if you can add some example values for what the fields in these config dataclasses should look like. At least at a glance, I wasn't sure about the differences between all these different config types and where/how they are supposed to be used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Mostly nits; nothing blocking
What's in this Change?
Adding a new pipeline stack base class. Will use in project applications