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

Canned mysql (2nd attempt) #107

Closed
wants to merge 7 commits into from

Conversation

borgoat
Copy link

@borgoat borgoat commented Oct 25, 2019

Ciao @gianarb !

I initially wanted to target your feature/canned-mysql branch but I ended up having to rebase to master...

I tried to align it based on the comments in #59 #98 #102 #105

Gianluca Arbezzano and others added 7 commits April 20, 2019 11:47
Reference testcontainers#19

I bootstrapped a mysql canned container.

```
ctx := context.Background()
c, err := MySQLContainer(ctx, MySQLContainerRequest{
        RootPassword: "root",
        Database:     "hello",
})
if err != nil {
        t.Fatal(err.Error())
}
defer c.Container.Terminate(ctx)
sqlC, err := c.GetDriver("root", "root", "hello")
if err != nil {
        t.Fatal(err.Error())
}
_, err = sqlC.ExecContext(ctx, "CREATE TABLE example ( id integer, data varchar(32) )")
if err != nil {
        t.Fatal(err.Error())
}
```

Signed-off-by: Gianluca Arbezzano <gianarb92@gmail.com>
@gianarb
Copy link
Member

gianarb commented Nov 20, 2020

Hello! At this point in time, I think we are not ready to onboard canned/modules as part of the main repository.
I am taking a step back here for now but you are free to list your own modules in our documentation if you decide to release it!

#268

Thanks a lot

@gianarb gianarb closed this Nov 20, 2020
@mdelapenya
Copy link
Member

We started this PR, which could be the seed for modules: #618

It creates examples to be consumed by the website, but turning them into modules would be a matter of adjusting the templates.

@mdelapenya
Copy link
Member

Given #618 has been merged, we created #636, where you can find how to contribute new modules :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants