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.

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

Merge main into live #41528

Merged
merged 6 commits into from
Jun 22, 2024
Merged

Merge main into live #41528

merged 6 commits into from
Jun 22, 2024

Commits on Jun 21, 2024

  1. capitalize rag (#41510)

    @gewarren
    gewarren committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    0a0c8de View commit details
    Browse the repository at this point in the history

  2. Fix misleading in the Main doc about async/Task (#41356) 

    * Add Main declarations

* Replace signature by declaration

* Access modifier rule

* Improve main overview

* Rework Task boilerplate code part

* Fix example

* Update main-command-line.md

* Remove indication about application isn't Library or Service

* fix spellcheck

Co-authored-by: Rich Lander <rlander@microsoft.com>

* Reverse column order in table

* Rever column order that show Main declaration

* fix missing empty line

* Apply suggestions from code review

Co-authored-by: David Pine <david.pine@microsoft.com>

* Specify 'any' explictit access modifier

Co-authored-by: Bill Wagner <wiwagn@microsoft.com>

* Remove boilerplate part

* Add info about completed task

* Fix link

* Remove trailing space

* Fix link

* Fix link

* Fix link

* Fix link

* Reset boilerplate

* Main can be internal

* Rework Main access modifier part

---------

Co-authored-by: Rich Lander <rlander@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: Bill Wagner <wiwagn@microsoft.com>
    @vernou @richlander
    @IEvangelist @BillWagner
    4 people committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    4281f27 View commit details
    Browse the repository at this point in the history

  3. Replace "standard" with "conventions" (#41520) 

    From anonymous feedback:

> This article should state that C# and .NET common coding conventions are not "Code Standards" and should not be treated as such.  Code standards would be ANSI, ECMA, ISO, standardized.The common .NET conventions work well for a broad extensive library of .NET API BCL calls but do not scale for a large complex business application.  The conventions work against aligning the code with the business processes by emphasizing deconstructing the C# code into a large complex web of short classes and method.Code should encapsulate business logic and algorithms.

Rather than address the purpose of different rulesets, I replaced all uses of "standard" with "convention". That reinforces that these are our conventions and any customer team should define their own. Their conventions may choose to use ours as a starting point, but shouldn't feel obligated to do so.
    @BillWagner
    BillWagner committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    b3be7f2 View commit details
    Browse the repository at this point in the history

  4. Remove note about struct keyword (#41519) 

    The introduction implies that Java allows `struct` types. It does not.

Fixes anonymous feedback.
    @BillWagner
    BillWagner committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    159e7f0 View commit details
    Browse the repository at this point in the history

  5. Code fence all escape sequences (#41521) 

    Fixes #40742
    @BillWagner
    BillWagner committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    f83f64c View commit details
    Browse the repository at this point in the history

  6. Prompt engineering updates (#41518) 

    Prompt engineering edits

---------

Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
    @alexwolfmsft @gewarren
    alexwolfmsft and gewarren committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    447a5a3 View commit details
    Browse the repository at this point in the history