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

Pull from origin #1

Merged
merged 98 commits into from
Feb 2, 2017
Merged

Pull from origin #1

merged 98 commits into from
Feb 2, 2017

Conversation

teyc
Copy link
Owner

@teyc teyc commented Feb 2, 2017

No description provided.

Chui Tey and others added 30 commits January 30, 2016 12:12
Unit tests were failing
…leTypes

if there is an nvarchar(max) in a table type, we should output the length as -1
Fix nvarchar(max) in table types, and escape user names
Allow specifying a connection string in the console
I’m trying to use SchemaZen for a database whose name contains a dot
(i.e. `MyDatabase.Dev`). Presently it fails because the DB names are
not being escaped (with square brackets) correctly. This fixes that.
# Conflicts:
#	model/Database.cs
Add support for table indexes with filters
marcio-santos-zocdoc and others added 28 commits May 12, 2016 12:46
# Conflicts:
#	model/Models/Database.cs
Created Schemazen library project
… added directly after the trigger definition with no new line.
…sn't tacked onto teh end of the definition when there are no SETs
Trigger script without SETs produces invalid script
…rigger

Enable Test "TestScriptViewInsteadOfTrigger"
* String interpolation in favor of string format or concatenation.
* Public fields converted to properties.
* Auto Initialized properties where applicable.
* Refactored SqlAssembly.ScriptCreate to be more readable.
* Renamed private fields to start with "_".
The tests explain the problem. Thanks @Nazgul07 for reporting.
@teyc teyc merged commit 6c3c213 into teyc:master Feb 2, 2017
teyc pushed a commit that referenced this pull request Jun 7, 2022
Update Column.cs to support RowGuidCol
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.

8 participants