Skip to content
This repository was archived by the owner on Oct 4, 2024. It is now read-only.

Conversation

@josjaf
Copy link
Contributor

@josjaf josjaf commented May 13, 2019

Issue #, if available:
Fixing bad hooks to gitlab
Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jaymccon jaymccon changed the base branch from master to develop May 14, 2019 15:37
@jaymccon
Copy link
Contributor

Thanks for the contribution @josjaf 😃

@jaymccon jaymccon merged commit dd6573f into aws-quickstart:develop May 14, 2019
@josjaf
Copy link
Contributor Author

josjaf commented May 14, 2019

No problem - let me know if you have any issues during testing - happy to help you work through them. At this point - I diverged from your code for my gitlab to S3, so it might take a bit of effort for me to stand this back up.

pedrojmendoza added a commit to pedrojmendoza/quickstart-git2s3 that referenced this pull request Jul 1, 2019
As part of aws-quickstart#33, a Python 3 only string formatting mechanism was introduced (https://realpython.com/python-f-strings/) but the CFN template is still using python2.7 as runtime for the lambdas hence a SyntaxError is happening
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.

2 participants