Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Label source-postgres to GA #15380

Merged
merged 5 commits into from
Aug 6, 2022
Merged

Label source-postgres to GA #15380

merged 5 commits into from
Aug 6, 2022

Conversation

grishick
Copy link
Contributor

@grishick grishick commented Aug 5, 2022

What

Change releaseStage to generally_available for source-postgres

@grishick grishick requested a review from a team as a code owner August 5, 2022 21:39
@github-actions github-actions bot added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels Aug 5, 2022
@grishick grishick temporarily deployed to more-secrets August 5, 2022 21:42 Inactive
@grishick grishick temporarily deployed to more-secrets August 5, 2022 22:08 Inactive
@tuliren
📄 Postgres source: fix links in the spec (#15382)
71d0f00 
* Add internal anchors back

* Remove internal anchor

* Use links for subtitles

* Remove one more anchor
@airbytehq airbytehq deleted a comment from grishick Aug 5, 2022
tuliren
tuliren approved these changes Aug 5, 2022
View reviewed changes
Copy link
Contributor

@tuliren tuliren left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The publish command has not been picked up by any action runner yet. I merged a PR to this branch that updated the broken links in the spec file.

@tuliren
Copy link
Contributor

tuliren commented Aug 5, 2022 •
edited by github-actions bot
Loading

/publish connector=connectors/source-postgres

🕑 Publishing the following connectors:
connectors/source-postgres
https://github.com/airbytehq/airbyte/actions/runs/2806670681


Connector Did it publish? Were definitions generated?
connectors/source-postgres ✅ ✅

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@tuliren
Copy link
Contributor

tuliren commented Aug 5, 2022 •
edited by github-actions bot
Loading

/publish connector=connectors/source-postgres-strict-encrypt

🕑 Publishing the following connectors:
connectors/source-postgres-strict-encrypt
https://github.com/airbytehq/airbyte/actions/runs/2806671612


Connector Did it publish? Were definitions generated?
connectors/source-postgres-strict-encrypt ✅ ❌

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@tuliren tuliren temporarily deployed to more-secrets August 5, 2022 22:53 Inactive
@tuliren tuliren merged commit d8dfcaa into master Aug 6, 2022
@tuliren tuliren deleted the greg/src-postgres-toga branch August 6, 2022 00:52
@michel-tricot
Copy link
Contributor

Woohoo!!

@octavia-squidington-iii octavia-squidington-iii mentioned this pull request Aug 10, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tuliren tuliren tuliren approved these changes

@misteryeo misteryeo misteryeo approved these changes

@edgao edgao Awaiting requested review from edgao

@subodh1810 subodh1810 Awaiting requested review from subodh1810

Assignees
No one assigned
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/postgres connectors/source/postgres-strict-encrypt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@grishick @tuliren @michel-tricot @misteryeo @octavia-squidington-iii