Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Added build instructions for newbies to Csharp. #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

topsrek
Copy link

@topsrek topsrek commented Apr 29, 2018

Hi!
This is the first time I contribute to open source projects, so please bear with me.
As there are quite few new features, I wanted, added since the last release, I wanted to build it myself.

After a bit of googling, I found out how to do it. To save time for the next one like me I added the instructions to the readme.

My compiler also failed on a "$" in the code. I deleted it and hope it is not necessary.

Cheers,
Topsrek

@@ -108,7 +108,7 @@ private void BrowseTo(string url)
webBrowser.Visible = true;
try
{
Debug.WriteLine($"Navigating: {url}");

Choose a reason for hiding this comment

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

That $ is for string interpolation. if you got a build error with it in that means you've not got the most recent .Net version selected as string interpolation was added in C# 6

https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/tokens/interpolated

(with string interpolation $"Navigating: {url}" is equivalent to "Navigating: " +url.toString() removing the $ changes the meaning to be the hardcoded and booring string: "Navigating: {url}"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants