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.

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

sftp binding component #3505

Merged
merged 18 commits into from
Oct 25, 2024
Merged

sftp binding component #3505

merged 18 commits into from
Oct 25, 2024

Conversation

arslanmusta
Copy link
Contributor

@arslanmusta arslanmusta commented Aug 7, 2024

Description

Sftp binding component implementation

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #3206

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
@arslanmusta arslanmusta marked this pull request as ready for review August 7, 2024 18:07
@arslanmusta arslanmusta requested review from a team as code owners August 7, 2024 18:07
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, it's interesting to see a new binding!

go.mod Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
bindings/sftp/sftp.go Outdated Show resolved Hide resolved
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
Signed-off-by: Mustafa Arslan <[email protected]>
bindings/sftp/sftp.go Show resolved Hide resolved
Signed-off-by: Mustafa Arslan <[email protected]>
Copy link

github-actions bot commented Oct 4, 2024

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale label Oct 4, 2024
@github-actions github-actions bot removed the stale label Oct 4, 2024
@berndverst berndverst added the documentation required This issue needs documentation label Oct 25, 2024
@berndverst
Copy link
Member

@arslanmusta next steps to make this a component:

  • Open a docs PR for the new components in github.com/dapr/docs
  • Work with dapr/dapr (runtime) maintainers to import the latest version of dapr/components-contrib and to register your new components. If the component isn't explicitly registered in dapr/dapr it essentially does not exist and cannot be used.

@berndverst berndverst merged commit ab9422d into dapr:main Oct 25, 2024
87 of 88 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation required This issue needs documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sftp binding
3 participants