OK, there are actually a few tests that still don't pass on my machine, but I'll fix them.
Is the change about the semicolons that are added to all the statement ok for you? Because I was not sure that it would not be considered a massive breaking change.

My thought is we should put together an ADR (Architecture Decision Record) for anISeparator interface. As I mentioned, I had changes for this in my local dev environment, just hadn't pushed it up yet. I think, taking a step back, I should have started with an ADR to explain / motivate the change.

I honestly don't know if the semicolons added to all the statements is OK or not, but my definition of OK would be if it doesn't break any consumers and is functionally the same, then it is "OK". Does that make sense?

In terms of what the ADR would talk about:

1. MySQL default is ; but supportsDELIMITER <char sequence> statement to override
2. Postgres supports "dollar quoting" e.g.

   $function$BEGIN RETURN ($1 ~ $q$[\t\r\n\v\\]$q$);END;$function$

3. Firebird default is ; but supportsSET TERM keyword phrase
4. DB2 default is ; but --#SET TERMINATOR phrase
5. Oracle SQLPlusset cmdsep phrase
6. Hana: In SAP HANA, the default statement delimiter for executing multiple SQL statements is a semicolon (;), but you can specify a different delimiter using the -c command-line option in HDBSQL.
7. T-SQL and SyBase: ; and GO for batches.
8. Redshift uses ;
9. and so on

That makes absolutely sense. I'm not sure about the dollar quoting though. Isn't it about IQuoter?
I think my changes may not go against what you started to work on, if that is about the ISeparator.
I was curious about what EF Core did with migration scaffolding, and saw that they use a single dependency for quotes and separators, what do you think about it?

@jzabroski All tests pass, what do you think about implementing the ISeparator in this pull request ?

@jzabroski All tests pass, what do you think about implementing the ISeparator in this pull request ?

I meant to reply last week but was pretty down with allergies. I will try tonight to push up to my jzabroski/FluentMigrator repo what I have so you can look at it.

I'm getting back to this today. Was pretty low on energy due to seasonal allergies the last two weeks, my apologies.

No worries ! Take care

@PhenX Plan to merge this today