Closed aldemirenes closed 4 years ago
I signed it!
Hi @chuwy ! I realized that you force-pushed to redshift-migrations branch. Since I worked on the old version of redshift-migrations branch, a lot of conflict happened in this PR after force-push. Should I rebase the commits in this PR to last version of the redshift-migrations branch ?
Thanks for the review @rzats ! I made changes according to your feedbacks. Also, I tried to rebase to the latest version of redshift-migrations branch however I could not compile to project due to some changes on the last experimental commits. Therefore, I rebased my branch to https://github.com/snowplow/iglu/commit/b389ae3aa24ca3244854a0a029c5fcc13cc51778. I can rebase to the latest version of the redshift-migrations branch after working on that branch end.
Hey @chuwy! Just a heads-up, I made necessary changes according to your comments!
These projects migrated to their own repos.
Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.