Thanks to visit codestin.com
Credit goes to github.com

Skip to content

plugin for Entity Framework Classic 7 #397

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 20, 2019
Merged

plugin for Entity Framework Classic 7 #397

merged 2 commits into from
Jul 20, 2019

Conversation

@JendaPerl
Copy link
Contributor

I've just made a copy of the Miniprofiler.EF6 as Miniprofiler.EFC7 and tweaked it to reference and use the https://entityframework-classic.net/ instead of the official EF 6. Not sure what needs to be done to get a nuget package, but the obtained MiniProfiler.EFC7.dll does work with the current Miniprofiler and EFC 7.1.9.

The initialization just needs to be changed to

StackExchange.Profiling.EntityFrameworkClassic7.MiniProfilerEFC7.Initialize();
in place of
StackExchange.Profiling.EntityFramework6.MiniProfilerEF6.Initialize();

I would very much appreciate if anyone could finish whatever needs to be done to end up with a nuget.

@JendaPerl
Copy link
Contributor Author

What the what?

no matching manifest for windows/amd64 10.0.17134 in the manifest list entries
##[warning]Docker pull failed with exit code 1, back off 3.444 seconds before retry.
##[command]C:\Program Files\Docker\docker.EXE pull redis
Using default tag: latest
no matching manifest for windows/amd64 10.0.17134 in the manifest list entries
latest: Pulling from library/redis
##[warning]Docker pull failed with exit code 1, back off 3.905 seconds before retry.
##[command]C:\Program Files\Docker\docker.EXE pull redis
Using default tag: latest
no matching manifest for windows/amd64 10.0.17134 in the manifest list entries
latest: Pulling from library/redis
##[error]Docker pull failed with exit code 1

It seems setting up the tests failed before anything I did came to play.

@NickCraver
Copy link
Member

I was getting netcoreapp3.0 initial bits working (which conflicted) - going to try and update some of these PRs to work :)

NickCraver
NickCraver approved these changes Jul 20, 2019
View reviewed changes
Copy link
Member

@NickCraver NickCraver left a comment

Choose a reason for hiding this comment

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

Looking good after a master merge - I'll try and get packages added in the next NuGet release.

Thank you for the contribution!

@NickCraver NickCraver merged commit b63599c into MiniProfiler:master Jul 20, 2019
@JendaPerl
Copy link
Contributor Author

I noticed that I've made a typo in the readme. I suggest install of Miniprofiler.EFC76 instead of Miniprofiler.EFC7. Could you fix this or change it to whatever you decide to call the package?
I think that's easier than going through the exercise of forking and creating a pull request for a one character change. :-)

Thanks!

@NickCraver
Copy link
Member

Done in c3c66d3 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NickCraver NickCraver NickCraver approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JendaPerl @NickCraver