Skip to content
This repository was archived by the owner on Aug 16, 2022. It is now read-only.

feat: Add tags for Glue Databases#1420

Closed
travwarren wants to merge 1 commit intocloudquery:mainfrom
travwarren:feature/glue-db-tags
Closed

feat: Add tags for Glue Databases#1420
travwarren wants to merge 1 commit intocloudquery:mainfrom
travwarren:feature/glue-db-tags

Conversation

@travwarren
Copy link
Copy Markdown

@travwarren travwarren commented Aug 13, 2022

Summary

Adds tags for Glue databases


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests. Learn more about testing here 🧪
  • Update the docs by running go run ./docs/docs.go and committing the changes 📃
  • Ensure the status checks below are successful ✅

@travwarren travwarren requested review from a team and irmatov and removed request for a team August 13, 2022 20:27
@erezrokah
Copy link
Copy Markdown
Member

Hi @travwarren and thank you for the PR 🎉 You've caught us in a middle of a migration to a monorepo, and the code for the AWS provider is now here.

I've recreated your PR (using your commits) in cloudquery/cloudquery#1326 so we can continue the work there. You can also recreate it yourself if you'd like

@erezrokah
Copy link
Copy Markdown
Member

Closing in favor of cloudquery/cloudquery#1326, we do any follow up work there.
Thanks for the contribution @travwarren 🚀

@erezrokah erezrokah closed this Aug 16, 2022
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