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

merge code from master to prtest #22

Merged
merged 46 commits into from
Sep 22, 2017
Merged

merge code from master to prtest #22

merged 46 commits into from
Sep 22, 2017

Conversation

OPSTestPPE
Copy link

Title

On the title describe
what you've fixed (or created) with this Pull Request (PR).

Summary

Insert a short (one or two sentence) summary here.

Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details

Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.

This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.

Suggested Reviewers

If you know who should review this, use '@' to request a review.

mairaw and others added 30 commits September 18, 2017 11:34
* added info about relative paths for output

* edits
Updated Cut to Copy in Copy/Cut and Paste this code into ...
* fixed formulas

* fixed formula

* fixed formula
…3171)

* what's new Tuples : sample use InvariantCulture

* vb tuple example fix new language feature sample

* minor typos

* compiler warning corrected to match previously mentioned number
* removed a "ghost" quote

* fixed inconsistent "read-only" and "read only"

* code formatting (missing space)
* Create csharp-version-history.md

* Added the actual text

* Updated TOC with my entry.

* first pass edit

1. Add Metadata
1. Change person
1. Formalize from blog post to docs style language.

* Update csharp-version-history.md

* fix build warnings on links.

* respond to feedback

* fixed build warnings

* fixed version capitalization
* First draft of new tutorial

Learn the structure of a C# program, and  explore the string class.

proofread and tighten language.

respond to feedback

2nd draft

Move more code earlier.
Tighten vocabulary discussions.

Update outline

start on the next draft.

add section on searching strings

fix build errors

finish the string tutorial.

reorder: ready for proofread.

proofread

update to use new syntax

Update Tutorial to use the new engine

typo in the title

* build test, try samples

* 2nd draft of the numbers tutorial.

* proofread the second tutorial and create the third

Branching and looping is ready for initial review.

* respond to feedback on first tutorial.

* proof read 3rd tutorial

And add links to other C# Guide content

* Remove links to upcoming tutorials.

* Respond to feedback.

* fix yaml parsing error.

* More feedback.

* feedback from team review

Configure ms.technology for all quick starts to be
devlang-csharp-interactive.
Update .NET home page and C# Guide home page to point to the
interactive tutorials.

* Describe booleans, and update metadata.

* add a tip about compiler messages

* update timings of tutorials, update tip text

* final review.
…#3185)

* Fix typo: AesCrytpServiceProvider -> AesCryptoServiceProvider

* Fix typo: crytopgraphy -> cryptography
* added net core behavior

* feedback

* feedback

* feedback
The link target was updated for the Quick Starts, but the displayed
text was not.
* Updating condition(s) under which the method throws

* Correcting a few hard-to-localize bits of content
aliozgur and others added 16 commits September 21, 2017 08:45
* Update mutually recursive functions snippet

Correct implementation should be something like (this one)[https://en.wikipedia.org/wiki/Mutual_recursion]

* Typo fix in function name

Event -> Even
* 494407 Corrections to DnsSafeHost and IdnHost

* Actioning all comments :)

* I used the wrong inline code delimiter
* update target framework to 2.0

* Improve Suits and Ranks enumerable definitions

* Revert "update target framework to 2.0"

This reverts commit 008d19d.
* Fixed Kelvin output, make culture independent

- Fixed wrong expected output for Kelvin in the sample usage
- Defined current culture in the sample usage to work out-of-the-box for international locales

* Remove obsolete full qualified name

* CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture
* Fixed Kelvin output, make culture independent

- Fixed wrong expected output for Kelvin in the sample usage
- Defined current culture in the sample usage to work out-of-the-box for international locales

* Remove obsolete full qualified name

* CurrentThread.CurrentCulture -> CultureInfo.CurrentCulture
For whatever reason, the sample XML was duplicated -- this commit removes that duplication so that the example is more readable.
Typo in document - IOne instead of One
Reported by @pauliusnorkus. Thanks to @dasMulli for reminding me about the framework problem here. Using `netstandard2.0` worked during pre-release but no more.
@OPSTestPPE OPSTestPPE merged commit c86c7bd into prtest Sep 22, 2017
luyajun0205 pushed a commit that referenced this pull request Apr 23, 2018
* added note regarding Path.Combine()

* Update how-to-write-text-to-a-file.md

* Update how-to-write-text-to-a-file.md
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.