Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

PR to demonstrate abstracted pipelines and new configuration interface #126

Closed
wants to merge 5 commits into from
Closed

PR to demonstrate abstracted pipelines and new configuration interface #126

wants to merge 5 commits into from

Conversation

turkenh
Copy link
Collaborator

@turkenh turkenh commented Oct 26, 2021

Description of your changes

Generate tf-aws using the new interface defined in crossplane/terrajet#128

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

See crossplane/terrajet#128

@turkenh
Copy link
Collaborator Author

turkenh commented Oct 27, 2021

@muvaf refactored the resource configuration interface as we discussed offline today, please see this commit here and this one in terrajet to get a sense on how it would look like.

@turkenh turkenh changed the title Abstracted pipelines PR to demonstrate abstracted pipelines and new configuration interface Oct 30, 2021
@muvaf
Copy link
Member

muvaf commented Nov 16, 2021

#128 includes commits from this PR.

@muvaf muvaf closed this Nov 16, 2021
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