Skip to content

Add composer installation documentation#1

Closed
chadrien wants to merge 1 commit into
DBDiff:masterfrom
chadrien:patch-1
Closed

Add composer installation documentation#1
chadrien wants to merge 1 commit into
DBDiff:masterfrom
chadrien:patch-1

Conversation

@chadrien

Copy link
Copy Markdown
Contributor

You'd need to submit the project to packagist first and then we could directly use it a a composer dependency.

@jasdeepkhalsa

Copy link
Copy Markdown
Member

Another great suggestion, will look into packagist and get this merged in

@jasdeepkhalsa

Copy link
Copy Markdown
Member

I needed to edit README.md a bit so have made the edit directly instead of merging in this PR. Thanks again for the suggestion! Your suggestion has landed in e18f3ca

@jasdeepkhalsa

Copy link
Copy Markdown
Member

Also if the package https://packagist.org/packages/jawbfl/dbdiff is yours on Packagist, it may be an idea to remove this so to not confuse people as to the whereabouts of the official source!

@chadrien chadrien deleted the patch-1 branch March 25, 2016 11:52
@ishmaelen ishmaelen mentioned this pull request Dec 28, 2016
jasdeepkhalsa added a commit that referenced this pull request Mar 25, 2026
AlterTableDropConstraintSQL::getUp() now throws a RuntimeException when
the constraint name is empty/null instead of emitting invalid SQL like
ALTER TABLE `t` DROP CONSTRAINT ``;

Added unit tests covering:
- Valid constraint name generates correct SQL
- getDown() restores the constraint via ADD
- Empty name throws RuntimeException
- Null name throws RuntimeException

Refs: bugs.md Bug #1
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.

2 participants