Skip to content

Create postgres driver#4

Open
MakoTano wants to merge 8 commits intomasterfrom
create_postgres_driver
Open

Create postgres driver#4
MakoTano wants to merge 8 commits intomasterfrom
create_postgres_driver

Conversation

@MakoTano
Copy link
Contributor

No description provided.

func init() {
core.RegisterDriver("mysql", &MySQL{})
// core.RegisterDriver("mysql", &MySQL{})
core.RegisterDriver("postgres", &Postgres{})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't have any idea to resolve this..

@MakoTano
Copy link
Contributor Author

MakoTano commented Aug 12, 2016

now I tested create command only.

@MakoTano
Copy link
Contributor Author

now I tested up and down commads.

@MakoTano
Copy link
Contributor Author

I tested sync command. Please review this!

@MakoTano MakoTano changed the title WIP - Create postgres driver Create postgres driver Aug 12, 2016
@kaneshin
Copy link
Member

Cool! I'll check it after prepared postgres environment.

@MakoTano MakoTano force-pushed the create_postgres_driver branch from 4bbf880 to 8dbd311 Compare September 15, 2016 10:39
@codecov-io
Copy link

codecov-io commented Sep 15, 2016

Current coverage is 41.42% (diff: 0.91%)

Merging #4 into master will decrease coverage by 7.26%

@@             master         #4   diff @@
==========================================
  Files            17         18     +1   
  Lines           608        717   +109   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            296        297     +1   
- Misses          289        397   +108   
  Partials         23         23          

Powered by Codecov. Last update 047f8b4...8dbd311

@MakoTano
Copy link
Contributor Author

@kaneshin 進化しすぎてて何が何やら・・・w

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.

3 participants