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

fix: fix column unsigned change in update #108

Merged
merged 6 commits into from
Mar 22, 2021

Conversation

oguzhanT
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Mar 22, 2021

Codecov Report

Merging #108 (322f84f) into master (e21cc6b) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #108      +/-   ##
============================================
+ Coverage     50.14%   50.25%   +0.10%     
- Complexity      500      501       +1     
============================================
  Files            17       17              
  Lines          1372     1379       +7     
============================================
+ Hits            688      693       +5     
- Misses          684      686       +2     

@Jeckerson
Copy link
Member

@oguzhanT Hello! Could you please add test case?

@Jeckerson Jeckerson added this to the 2.1.x milestone Mar 22, 2021
@Jeckerson Jeckerson linked an issue Mar 22, 2021 that may be closed by this pull request
@Jeckerson Jeckerson merged commit f0f76e1 into phalcon:master Mar 22, 2021
@Jeckerson Jeckerson added this to In Progress in Phalcon ecosystem board via automation Mar 23, 2021
@Jeckerson Jeckerson moved this from In Progress to Done in Phalcon ecosystem board Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[BUG] change column unsigned didnt work
2 participants