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

azure functions or csharp microservice-console-stream consumer or (ev… #2224

Merged
merged 2 commits into from
Apr 6, 2020
Merged

azure functions or csharp microservice-console-stream consumer or (ev… #2224

merged 2 commits into from
Apr 6, 2020

Conversation

dzmitry-lahoda
Copy link

…en more likely) fsharp microservice-console-stream consumer

  • are REAL, and will not put codes into separate files
  • i read EF tutorial for EF, not for good practices (which are not actually universally good)
  • please keep tentative practices out of core tutorials
  • may create separate guideline-best practices page for that

…en more likely) fsharp microservice-console-stream consumer

- are REAL, and will not put codes into separate files
- i read EF tutorial for EF, not for `good practices` (which are not actually universally good)
- please keep tentative practices out of core tutorials
- may create separate guideline-best practices page for that
Copy link
Member

@ajcvickers ajcvickers left a comment

Choose a reason for hiding this comment

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

@dzmitry-lahoda I get where you are coming from here, but suggesting a slightly different wording.

entity-framework/core/get-started/index.md Outdated Show resolved Hide resolved
@ajcvickers ajcvickers merged commit 2d8c6bc into dotnet:master Apr 6, 2020
roji pushed a commit that referenced this pull request Apr 27, 2020
#2224)

* Update entity-framework/core/get-started/index.md

Co-authored-by: Arthur Vickers <ajcvickers@hotmail.com>
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.

3 participants