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 #342 - Adding the support of altering user options #345

Merged
merged 1 commit into from
Aug 17, 2021

Conversation

iifawzi
Copy link
Contributor

@iifawzi iifawzi commented Aug 17, 2021

Hi, this PR should add the support of altering user options, not all the options are included though, because I'm not aware of all of them, I've just introduced the base.

Fixes #342
Signed-off-by: Fawzi E. Abdulfattah iifawzie@gmail.com

@iifawzi iifawzi force-pushed the supporting-user-alter branch 4 times, most recently from 4e66de5 to 9e88376 Compare August 17, 2021 17:54
@codecov
Copy link

codecov bot commented Aug 17, 2021

Codecov Report

Merging #345 (2eff24a) into QA (f6723ea) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##                 QA     #345   +/-   ##
=========================================
  Coverage     99.80%   99.80%           
- Complexity     1915     1916    +1     
=========================================
  Files            63       63           
  Lines          4602     4604    +2     
=========================================
+ Hits           4593     4595    +2     
  Misses            9        9           
Impacted Files Coverage Δ
src/Components/AlterOperation.php 100.00% <ø> (ø)
src/Statements/AlterStatement.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6723ea...2eff24a. Read the comment docs.

@iifawzi iifawzi changed the title Adding the support of altering user statements Adding the support of altering user options Aug 17, 2021
@williamdes williamdes added this to the 4.7.3 milestone Aug 17, 2021
Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

This looks good, to better understand I will cut out and merge into QA and master your newly added tests

@williamdes williamdes self-assigned this Aug 17, 2021
williamdes pushed a commit that referenced this pull request Aug 17, 2021
Pull-request: #345

Signed-off-by: Fawzi E. Abdulfattah <iifawzie@gmail.com>
Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

Awesome !
I really like the diff it makes on master branch out files

Would you mind rebasing please ?

Signed-off-by: Fawzi E. Abdulfattah <iifawzie@gmail.com>
Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

🚢

@williamdes williamdes changed the title Adding the support of altering user options Fix #342 - Adding the support of altering user options Aug 17, 2021
@williamdes williamdes merged commit a20e660 into phpmyadmin:QA Aug 17, 2021
williamdes added a commit that referenced this pull request Aug 17, 2021
Pull-request: #345

Signed-off-by: William Desportes <williamdes@wdes.fr>
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.

None yet

2 participants