Skip to content

Mysql#85

Merged
jssuzanne merged 55 commits into
AnyBlok:masterfrom
jssuzanne:mysql
Jun 21, 2019
Merged

Mysql#85
jssuzanne merged 55 commits into
AnyBlok:masterfrom
jssuzanne:mysql

Conversation

@jssuzanne
Copy link
Copy Markdown
Member

No description provided.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.02%) to 92.377% when pulling c9eafaa on jssuzanne:mysql into 00c3827 on AnyBlok:master.

@coveralls
Copy link
Copy Markdown

coveralls commented May 9, 2019

Coverage Status

Coverage decreased (-0.02%) to 92.379% when pulling 0335cf0 on jssuzanne:mysql into 00c3827 on AnyBlok:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.02%) to 92.377% when pulling c9eafaa on jssuzanne:mysql into 00c3827 on AnyBlok:master.

@petrus-v
Copy link
Copy Markdown
Contributor

petrus-v commented May 9, 2019

Great ! testing anyblok against mysql ! looks like it'll require some extra effort to fix some tests

👏 for that effort

Comment thread anyblok/tests/test_column.py Outdated
test = registry.Test.insert(col=now)
Test = registry.Test
import ipdb
ipdb.set_trace()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't feel you wanted to include it in you commit ;)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok forgot me I jump to a commit instead reading the whole diff

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it was a bad commit

@jssuzanne jssuzanne merged commit cf2f484 into AnyBlok:master Jun 21, 2019
@jssuzanne jssuzanne deleted the mysql branch June 21, 2019 07:35
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