Skip to content

Added the skeleton projects for EF#5607

Merged
johnsimons merged 6 commits into
masterfrom
john/ef_projects
Jul 16, 2026
Merged

Added the skeleton projects for EF#5607
johnsimons merged 6 commits into
masterfrom
john/ef_projects

Conversation

@johnsimons

Copy link
Copy Markdown
Member

These PR adds the initial skeleton projects for EF.

These persisters are not yet enabled for public consumption.

@@ -0,0 +1,25 @@
{
"Name": "PostgreSQL",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should Sql vs SQL casing be standardised on Sql?
Note that we already use PostgreSql elsewhere.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixed


protected abstract EFPersisterSettings CreateSettings(string connectionString);

static T GetRequiredSetting<T>(SettingsRootNamespace settingsRootNamespace, string key)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The raven persister makes these utility functions private functions in CreateSettings, is it worth being consistent?

@johnsimons johnsimons marked this pull request as ready for review July 16, 2026 03:51
@johnsimons johnsimons enabled auto-merge July 16, 2026 04:13
@johnsimons johnsimons merged commit 05334ed into master Jul 16, 2026
33 checks passed
@johnsimons johnsimons deleted the john/ef_projects branch July 16, 2026 04:19
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.

4 participants