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

Docker Shell Plugin #300

Closed
wants to merge 1 commit into from

Conversation

roy9495
Copy link
Contributor

@roy9495 roy9495 commented Jun 19, 2023

Overview

Docker Shell Plugin Created

Type of change

  • Created a new plugin
  • Improved an existing plugin
  • Fixed a bug in an existing plugin
  • Improved contributor utilities or experience

Related Issue(s)

  • Resolves: #
  • Relates: #

How To Test

Changelog

@AndyTitu AndyTitu added the in-progress this PR is being worked on/comments are in the process of being addressed by the contributor label Jun 20, 2023
@AndyTitu AndyTitu marked this pull request as draft June 20, 2023 14:33
@AndyTitu
Copy link
Contributor

Converting to draft as this is still Work In Progress

@arunsathiya
Copy link
Contributor

Hi @roy9495, I believe this submission is for the Hashnode Hackathon. There's ongoing work for a Docker shell plugin here from @itsCheithanya:

May I suggest that you two collaborate to get this to the finish line? We will be counting this contribution for both of you for the hackathon.

Please let me know if that is okay with you and whether I could help you two connect, on Discord or the 1Password Developer Workspace on Slack!

}

var defaultEnvVarMapping = map[string]sdk.FieldName{
"DOCKERHUB_TOKEN": fieldname.Token, // TODO: Check if this is correct
Copy link
Contributor

Choose a reason for hiding this comment

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

@roy9495 Are you sure DOCKERHUB_TOKEN exists and is interpreted by Docker CLI? I couldn't get this to work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No sir, I am not sure but here it was mentioned. So, I used it. I could not set it up on my PC, so I set it up on Gitpod.

@roy9495
Copy link
Contributor Author

roy9495 commented Jun 28, 2023

Hi @roy9495, I believe this submission is for the Hashnode Hackathon. There's ongoing work for a Docker shell plugin here from @itsCheithanya:

May I suggest that you two collaborate to get this to the finish line? We will be counting this contribution for both of you for the hackathon.

Please let me know if that is okay with you and whether I could help you two connect, on Discord or the 1Password Developer Workspace on Slack!
Yes, this submission is for the Hashnode Hackathon.
Ok , if @itsCheithanya is ok with it then it's ok with me

@techcraver
Copy link
Contributor

Also don't forget to submit your Hashnode blog post with the 1Password and BuildWith1Password tags. (Hashnode tags - just #1Password in the blog post text. The full instructions are here.

@roy9495
Copy link
Contributor Author

roy9495 commented Jun 28, 2023

@techcraver Ok sir, I will surely do that.
Thank you.

@itsCheithanya
Copy link
Contributor

Hi @roy9495, I believe this submission is for the Hashnode Hackathon. There's ongoing work for a Docker shell plugin here from @itsCheithanya:

* [Add support for docker cli login using Username and Email plugin #301](https://github.com/1Password/shell-plugins/pull/301)

May I suggest that you two collaborate to get this to the finish line? We will be counting this contribution for both of you for the hackathon.

Please let me know if that is okay with you and whether I could help you two connect, on Discord or the 1Password Developer Workspace on Slack!

Sure let's collaborate

@hculea
Copy link
Member

hculea commented Jul 5, 2023

Hey @roy9495, just wanted to give you an update about this - I am going to close this PR, since currently Cheithanya's PR (#301) is in a more mature state, so you can both pick up from there, at a later point. 😄

That being said, we figured that Docker is a tool that will involve some internal changes for the plugin ecosystem to support, so we're not proceeding with this plugin for the time being.

Thank you for your contribution, and we welcome you to contribute new plugins to the ecosystem in the future! ❤️

@hculea hculea closed this Jul 5, 2023
@roy9495
Copy link
Contributor Author

roy9495 commented Jul 5, 2023

Ok sir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress this PR is being worked on/comments are in the process of being addressed by the contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants