Skip to content
This repository was archived by the owner on Jun 30, 2025. It is now read-only.

fix(issue-351): implementation#359

Open
Maarc-D wants to merge 9 commits into
hashicorp:mainfrom
Maarc-D:fix/issue-351
Open

fix(issue-351): implementation#359
Maarc-D wants to merge 9 commits into
hashicorp:mainfrom
Maarc-D:fix/issue-351

Conversation

@Maarc-D

@Maarc-D Maarc-D commented Sep 1, 2022

Copy link
Copy Markdown

Description

Related to #351

@Maarc-D

Maarc-D commented Sep 1, 2022

Copy link
Copy Markdown
Author

Creation Ok, but on next plan want to remove password, need to check that

@Maarc-D Maarc-D changed the title WIP: fix(issue-351): initialization fix(issue-351): implementation Sep 1, 2022
@Maarc-D

Maarc-D commented Sep 5, 2022

Copy link
Copy Markdown
Author

Creation Ok and no more change on next launch. everything ok with this implementation

@ivorsmorenburg

Copy link
Copy Markdown

merge it

@Maarc-D

Maarc-D commented Nov 2, 2022

Copy link
Copy Markdown
Author

Used since then, no issue. Do you have a release date in mind ?

Comment thread internal/provider/resource_user.go Outdated
Co-authored-by: Ricardo Graça <81819036+ricardograca-scratch@users.noreply.github.com>
@ricardograca-scratch

Copy link
Copy Markdown

Hey @Maarc-D I'm just a regular user of this provider. I don't think me reviewing it will make any difference whether it's accepted or not.

@Maarc-D

Maarc-D commented Dec 5, 2022

Copy link
Copy Markdown
Author

Hey @Maarc-D I'm just a regular user of this provider. I don't think me reviewing it will make any difference whether it's accepted or not.

I gave it an attempt :p

@Maarc-D

Maarc-D commented Dec 20, 2022

Copy link
Copy Markdown
Author

Hey @Maarc-D I'm just a regular user of this provider. I don't think me reviewing it will make any difference whether it's accepted or not.

I gave it an attempt :p

@appilon @SarahFrench @bflad @megan07 is it possible to review / merge it please ? ;)

@rek

rek commented Jan 16, 2023

Copy link
Copy Markdown

Would love to get this reviewed/merged in.

@Maarc-D

Maarc-D commented Nov 9, 2023

Copy link
Copy Markdown
Author

Hey @Maarc-D I'm just a regular user of this provider. I don't think me reviewing it will make any difference whether it's accepted or not.

can you try, maybe it will make it move next step :p

@ricardograca-scratch

Copy link
Copy Markdown

I'm not a go developer and also it explicitly says "Code owner review required", and I'm not a code owner, but I'll try to get some traction on this issue.

@Maarc-D

Maarc-D commented Dec 1, 2023

Copy link
Copy Markdown
Author

I'm not a go developer and also it explicitly says "Code owner review required", and I'm not a code owner, but I'll try to get some traction on this issue.

I know but you seems to be "requested for review" and I cannot change or remove that :/

@ricardograca-scratch ricardograca-scratch left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me, but I'm not a Go developer nor a maintainer of this project, so I'm only writing this because there doesn't seem to be a way to not provide a review after having been added as a reviewer 🤷

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.

4 participants