Testing PR Comments. #23

grokys · 2017-04-20T18:48:32Z

No description provided.

grokys

Dud a review!

grokys · 2017-04-20T18:48:53Z

PullRequestSandbox/TestClass.cs

+{
+    class TestClass
+    {
+        public int PropertyWithComment1 { get; set; }


Comment 1 here.

Why are there more comments here than on .com?

Inline HTML snippet:

<!DOCTYPE HTML> <html> <head> <title>Title of the document</title> </head> <body> The content of the document...... </body> </html>

grokys · 2017-04-20T18:49:06Z

PullRequestSandbox/TestClass.cs

+        public int PropertyWithComment1 { get; set; }
+        public int Property1 { get; }
+        public int Property2 { get; }
+        public int PropertyWithComment2 { get; set; }


Comment 2 here.

This isn't a very good comment IMO.

Actually, it's kinda growing on me...

grokys · 2017-04-20T18:49:14Z

PullRequestSandbox/TestClass.cs

+            // Method1 method contents.
+        }
+
+        public void MethodWithComment2()


Comment 3 here.

Foo 3 here.

Bar 3 here.

grokys · 2017-04-26T11:11:14Z

PullRequestSandbox/Program.cs

    {
        static void Main(string[] args)
        {
-            /// yepyep


Comment on deleted line

grokys · 2017-05-03T20:47:57Z

PullRequestSandbox/TestClass.cs

+            // MethodWithComment2 method contents.
+        }
+
+        public void Method1()


On the right line this time?

grokys · 2017-05-04T17:01:03Z

PullRequestSandbox/TestClass.cs

+
+        public void Method1()
+        {
+            // Method1 method contents.


Comment here?

grokys · 2017-05-18T12:47:25Z

PullRequestSandbox/Program.cs


 namespace PullRequestSandbox
 {
    class Program


Comment on an old commit.

grokys · 2017-06-09T11:34:42Z

PullRequestSandbox/Program.cs

+        // Made a change.
        static void Main(string[] args)
        {
-            /// yepyep


Comment on a deleted line?

grokys · 2017-06-12T11:41:57Z

PullRequestSandbox/Program.cs

    class Program
    {
+        // Made a change.
        static void Main(string[] args)


grokys · 2017-06-19T12:17:39Z

PullRequestSandbox/TestClass.cs

+        public int PropertyWithComment2 { get; set; }
+        public int Property2 { get; }
+
+        public void MethodWithComment2()


Leaving a comment.

grokys · 2017-06-27T20:24:54Z

PullRequestSandbox/TestClass.cs

+        public void MethodWithComment2()
+        {
+            // MethodWithComment2 method contents.
+        }


Testing a new comment here.

grokys · 2017-06-28T20:22:04Z

PullRequestSandbox/Program.cs


 namespace PullRequestSandbox
 {
    class Program


Program comment

grokys · 2017-06-28T20:23:20Z

PullRequestSandbox/Program.cs

-            /// yepyep
        }
    }
 }


grokys · 2017-07-07T09:28:41Z

PullRequestSandbox/Program.cs

 namespace PullRequestSandbox
 {
    class Program
    {


Adding a comment with inline code block

Fenced code block

grokys · 2017-07-12T16:02:47Z

PullRequestSandbox/Program.cs

        static void Main(string[] args)
        {
-            /// yepyep
        }


jcansdale · 2017-07-25T15:31:21Z

PullRequestSandbox/Program.cs

 {
    class Program
    {
+        // Made a change.


need a green glyph

here's a blue one.

I wanted green!

grokys · 2017-08-04T14:12:14Z

PullRequestSandbox/TestClass.cs

+
+namespace InlineComments
+{
+    class TestClass


Testing a comment with markdown

A code block

Header 1

A checkbox

A checked checkbox

grokys · 2017-08-25T08:01:02Z

PullRequestSandbox/Program.cs

+        // Made a change.
+        // Another change
        static void Main(string[] args)
        {


Comment from fork.

grokys · 2017-10-13T13:17:08Z

PullRequestSandbox/TestClass.cs

+            // Method1 method contents..
+        }
+
+        public void AnotherMethod()


Commen on AnotherMethod()

grokys · 2017-10-16T09:51:03Z

PullRequestSandbox/Program.cs

    {
+        // Made a change.
+        // Another change
        static void Main(string[] args)


grokys · 2017-10-16T09:53:12Z

PullRequestSandbox/TestClass.cs

+    {
+        public int PropertyWithComment1 { get; set; }
+        public int Property1 { get; }
+        public int PropertyWithComment2 { get; set; }


grokys · 2017-11-07T08:47:48Z

PullRequestSandbox/Program.cs

        {
-            /// yepyep
        }
    }


Added test class for review comments.

ea34401

grokys commented Apr 20, 2017

View reviewed changes

grokys and others added 3 commits April 20, 2017 20:50

Made some changes.

3398937

Deleted line

12ce549

Merge branch 'master' into comments-test

1838602

grokys commented Apr 26, 2017

View reviewed changes

PullRequestSandbox/Program.cs

{

static void Main(string[] args)

{

/// yepyep

Copy link

Owner Author

grokys Apr 26, 2017

Choose a reason for hiding this comment

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

Comment on deleted line

Made a change

80eb07f

grokys commented May 3, 2017

View reviewed changes

PullRequestSandbox/TestClass.cs

// MethodWithComment2 method contents.

}

public void Method1()

Copy link

Owner Author

grokys May 3, 2017

Choose a reason for hiding this comment

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

On the right line this time?

grokys commented May 4, 2017

View reviewed changes

grokys commented May 18, 2017

View reviewed changes

PullRequestSandbox/Program.cs

namespace PullRequestSandbox

{

class Program

Copy link

Owner Author

grokys May 18, 2017

Choose a reason for hiding this comment

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

Comment on an old commit.

grokys commented Jun 9, 2017

View reviewed changes

PullRequestSandbox/Program.cs

// Made a change.

static void Main(string[] args)

{

/// yepyep

Copy link

Owner Author

grokys Jun 9, 2017

Choose a reason for hiding this comment

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

Comment on a deleted line?

Hard breakfast.

2ddd2b5

grokys commented Jun 12, 2017

View reviewed changes

PullRequestSandbox/Program.cs

class Program

{

// Made a change.

static void Main(string[] args)

Copy link

Owner Author

grokys Jun 12, 2017

Choose a reason for hiding this comment

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

Yep

grokys commented Jun 19, 2017

View reviewed changes

grokys commented Jun 27, 2017

View reviewed changes

grokys commented Jun 28, 2017

View reviewed changes

PullRequestSandbox/Program.cs

namespace PullRequestSandbox

{

class Program

Copy link

Owner Author

grokys Jun 28, 2017

Choose a reason for hiding this comment

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

Program comment

grokys commented Jun 28, 2017

View reviewed changes

PullRequestSandbox/Program.cs

/// yepyep

}

}

}

Copy link

Owner Author

grokys Jun 28, 2017

Choose a reason for hiding this comment

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

Foobar

grokys commented Jul 7, 2017

View reviewed changes

grokys commented Jul 12, 2017

View reviewed changes

PullRequestSandbox/Program.cs

static void Main(string[] args)

{

/// yepyep

}

Copy link

Owner Author

grokys Jul 12, 2017

Choose a reason for hiding this comment

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

Here?

grokys added 2 commits July 14, 2017 16:40

Hard live

edbb23b

Changed that.

38bfdc7

jcansdale reviewed Jul 25, 2017

View reviewed changes

grokys commented Aug 4, 2017

View reviewed changes

grokys mentioned this pull request Aug 18, 2017

Inline comments does not render code snippets github/VisualStudio#1152

Open

grokys and others added 2 commits August 18, 2017 12:50

Outdated a comment.

fe0cb88

Made another change

5affff8

grokys commented Aug 25, 2017

View reviewed changes

PullRequestSandbox/Program.cs

// Made a change.

// Another change

static void Main(string[] args)

{

Copy link

Owner Author

grokys Aug 25, 2017

Choose a reason for hiding this comment

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

Comment from fork.

grokys commented Oct 13, 2017

View reviewed changes

PullRequestSandbox/TestClass.cs

// Method1 method contents..

}

public void AnotherMethod()

Copy link

Owner Author

grokys Oct 13, 2017

Choose a reason for hiding this comment

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

Commen on AnotherMethod()

grokys commented Oct 16, 2017

View reviewed changes

PullRequestSandbox/Program.cs

{

// Made a change.

// Another change

static void Main(string[] args)

Copy link

Owner Author

grokys Oct 16, 2017

Choose a reason for hiding this comment

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

line

grokys commented Oct 16, 2017

View reviewed changes

grokys commented Nov 7, 2017

View reviewed changes

PullRequestSandbox/Program.cs

{

/// yepyep

}

}

Copy link

Owner Author

grokys Nov 7, 2017

Choose a reason for hiding this comment

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

dfg

Change.

13d7c52

jcansdale mentioned this pull request Jun 13, 2019

ArgumentException when opening PR file with comments github/VisualStudio#2385

Open

Testing PR Comments. #23

Are you sure you want to change the base?

Testing PR Comments. #23

Uh oh!

Conversation

grokys commented Apr 20, 2017

Uh oh!

grokys left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Header 1

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment