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

Remove deprecated connector dependency report #18889

Merged
merged 11 commits into from
Nov 3, 2022
Merged

Remove deprecated connector dependency report #18889

merged 11 commits into from
Nov 3, 2022

Conversation

tuliren
Copy link
Contributor

@tuliren tuliren commented Nov 3, 2022 •
edited
Loading

What

@github-actions github-actions bot added the area/connectors Connector related issues label Nov 3, 2022
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 03:59 Inactive
@github-actions github-actions bot removed the area/connectors Connector related issues label Nov 3, 2022
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 04:02 Inactive
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 04:06 Inactive
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 04:10 Inactive
@github-actions github-actions bot added the area/connectors Connector related issues label Nov 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2022 •
edited
Loading

Affected Connector Report

The latest commit has removed all connector-related changes. There are no more dependent connectors for this PR.

@github-actions github-actions bot removed the area/connectors Connector related issues label Nov 3, 2022
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 04:14 Inactive
@airbytehq airbytehq deleted a comment from github-actions bot Nov 3, 2022
@tuliren tuliren marked this pull request as ready for review November 3, 2022 04:19
@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 04:19 Inactive
erohmensing
erohmensing approved these changes Nov 3, 2022
View reviewed changes
Copy link
Contributor

@erohmensing erohmensing left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for tackling this!

.github/workflows/report-connectors-dependency.yml Outdated Show resolved Hide resolved
.github/comment_templates/connector_dependency_template.md Outdated Show resolved Hide resolved
.github/comment_templates/connector_dependency_template.md Outdated Show resolved Hide resolved
.github/workflows/report-connectors-dependency.yml Outdated Show resolved Hide resolved
@tuliren
Copy link
Contributor Author

tuliren commented Nov 3, 2022

@erohmensing, thank you very much.

@tuliren tuliren temporarily deployed to more-secrets November 3, 2022 16:11 Inactive
@tuliren tuliren merged commit a5f72d2 into master Nov 3, 2022
@tuliren tuliren deleted the liren/update-dependency-report-action branch November 3, 2022 16:31
@github-actions github-actions bot mentioned this pull request Nov 15, 2022
Closed
@octavia-squidington-iii octavia-squidington-iii mentioned this pull request Nov 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@erohmensing erohmensing erohmensing approved these changes

Assignees
No one assigned
Labels
connectors/source/postgres
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tuliren @erohmensing @octavia-squidington-iv