Skip to content

Comments

Edited README via GitHub#1

Closed
unoju wants to merge 1 commit intooctocat:masterfrom
unoju:patch-1
Closed

Edited README via GitHub#1
unoju wants to merge 1 commit intooctocat:masterfrom
unoju:patch-1

Conversation

@unoju
Copy link

@unoju unoju commented May 9, 2011

No description provided.

@octocat octocat closed this May 18, 2011
@masonzou
Copy link

test

@198103292005021004
Copy link

affirmative

@siekier
Copy link

siekier commented Feb 18, 2013

hi

@yzwork
Copy link

yzwork commented Feb 19, 2013

hello :)

@vamshimolleti
Copy link

a new comment

@ahmadjoon
Copy link

Hi!there!

@damontenegrod
Copy link

Hi

2013/4/29 ahmadjoon notifications@github.com

Hi!there!


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-17204510
.

Saludos,

David Montenegro

@ghost
Copy link

ghost commented May 3, 2013

HI

@souker
Copy link

souker commented May 3, 2013

hi

@ghost
Copy link

ghost commented May 3, 2013

Pre-me

On May 2, 2013, at 11:57 PM, souker notifications@github.com wrote:

hi


Reply to this email directly or view it on GitHub.

@heliogabalo
Copy link

hello world

@ghost
Copy link

ghost commented May 4, 2013

Thats my doctor ni mattter how hard he will never get it

Sent from my iPhone

On May 3, 2013, at 8:15 PM, heliogabalo santos jugon notifications@github.com wrote:

hello world


Reply to this email directly or view it on GitHub.

@coroa
Copy link

coroa commented Dec 16, 2013

Foo

1 similar comment
@coroa
Copy link

coroa commented Dec 16, 2013

Foo

@damontenegrod
Copy link

@damontenegrod
Copy link

Jobandtalent mail

/* Resets */
.ReadMsgBody { width: 100%; background-color: #f4f4f4;}
.ExternalClass {width: 100%; background-color: #f4f4f4;}
.ExternalClass, .ExternalClass p, .ExternalClass span, .ExternalClass font, .ExternalClass td, .ExternalClass div {line-height:100%;}
body {-webkit-text-size-adjust:none; -ms-text-size-adjust:none;}
body {margin:0; padding:0;}
table {border-spacing:0;}
table td {border-collapse:collapse;}
.yshortcuts a {border-bottom: none !important;}

/* Some sensible defaults for images */
img {outline:none; text-decoration:none; -ms-interpolation-mode: bicubic;}
a img {border:none;}

#notify_of_course_offer .smaller.main_heading strong:first-child {
  text-transform:capitalize;
}

/* Constrain email width for small screens */
@media screen and (max-width: 700px) {
  table[class="container"] {
    width: 100% !important;
  }

  #notify_of_course_offer .course_info {
    padding:10px!important;
  }

  #notify_of_course_offer .course_info .sale {
    font-size:20px!important;
    display:inline-block!important;
  }

  #notify_of_course_offer .course_info .current_price {
    font-size:23px!important;
    display:inline-block!important;
    padding:0!important;
    margin:10px 10px 0 0!important;
  }

  #notify_of_course_offer .discount_cell,
  #notify_of_course_offer .sales_cell {
    padding:0 10px!important;
  }

  #notify_of_course_offer .course_info .old_price {
    padding-left:10px!important;
    white-space:pre!important;
    font-size:17px!important;
  }

  #notify_of_course_offer .course_info .old_price:after {
    content:'\A';
  }

  #notify_of_course_offer .new_cell {
    padding-left:10px!important;
  }

  #notify_of_course_offer .course_info .percent,
  #notify_of_course_offer .course_info .saved {
    font-size:22px!important;
  }

}

/* Give content more room on mobile */
@media screen and (max-width: 480px) {
  td[class="container-padding"] {
    padding-left: 12px !important;
    padding-right: 12px !important;
  }
}

/* Styles for forcing columns to rows */
@media only screen and (max-width : 550px) {

  /* force container columns to (horizontal) blocks */
  td[class="force-col"] {
      display: block;
      padding-right: 0 !important;
  }
  table[class="col-2"] {
      /* unset table align="left/right" */
      float: none !important;
      width: 100% !important;
  }

  .band td {
    text-align:center!important;
  }

  .band .button {
    margin:0 auto!important;
    margin-bottom: 10px!important;
  }

  #notify_of_course_offer .buttons_container .force-col,
  #job_opening_invitation .buttons_container .force-col {
    width:100%!important;
  }

  #notify_of_course_offer .buttons_container .button,
  #job_opening_invitation .buttons_container .button {
    padding:0!important;
    width:100%!important;
    height:57px!important;
    font-size:21px!important;
    line-height:57px!important;
  }

  #notify_candidate_application .attached_cv_cell {
    padding-top:20px!important;
  }

  #notify_candidate_application .attached_cv {
    margin:0 auto!important;
    width:80%!important;
  }

  #notify_candidate_application .attached_cv .text_cell {
    vertical-align:middle!important;
  }

  #notify_candidate_application .attached_cv strong {
    display:inline!important;
  }

  #invite .top_companies_hiring .companies_logos .last_img {
    margin:20px 0 0!important;
  }

}






























Your friends have unlock exclusive job openings in
jobandtalent, what are you waiting for?

We help you to progress in your profesional career by delivering tailored job opportunities and online courses to boost your career

Accept the invitation

Top companies hiring right now
Discover companies with open positions and apply now

































                        You are receiving this email because you registered at Jobandtalent.com (3rd floor, 36-37 Featherstone Street London EC1Y 8QZ) and agreed to receive email from us.

@arturo-c
Copy link

a new comment

5 similar comments
@sohilpandya
Copy link

a new comment

@sohilpandya
Copy link

a new comment

@Voilak
Copy link

Voilak commented Oct 27, 2015

a new comment

@makokhavictor
Copy link

a new comment

@makokhavictor
Copy link

a new comment

@@ -1 +1,6 @@
Hello World! No newline at end of file
Hello World!
$ mkdir ~/Hello-WorldCreates a directory for your project called "Hello-World" in your user directory

Choose a reason for hiding this comment

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

hi

Copy link

Choose a reason for hiding this comment

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

Hello

Choose a reason for hiding this comment

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

Great observation! 👍 Testing with unicode and emojis: café résumé naïve

@szymonzo
Copy link

szymonzo commented Jun 2, 2016

a new comment

@szymonzo
Copy link

szymonzo commented Jun 2, 2016

a

@szymonzo
Copy link

szymonzo commented Jun 2, 2016

c

@szymonzo
Copy link

szymonzo commented Jun 2, 2016

a

$ mkdir ~/Hello-WorldCreates a directory for your project called "Hello-World" in your user directory
$ cd ~/Hello-WorldChanges the current working directory to your newly created directory
$ git initSets up the necessary Git files
Initialized empty Git repository in /Users/your_user_directory/Hello-World/.git/

Choose a reason for hiding this comment

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

Lets create another comment and see the position.

Copy link

Choose a reason for hiding this comment

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

I dont agree to this comment, so i will reply.

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

I agree with your point here.

This is a multi-line reply:

  • Point 1
  • Point 2
  • Point 3

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a very long text.

2 similar comments
@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a very long text.

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a very long text.

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with emojis 😊🚀👍

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a URL: https://www.example.com

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a mention to a user: @octocat

@YF-2023
Copy link

YF-2023 commented Feb 19, 2026

Testing comment with a quote:

This is a quoted text.

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

@jallurivardhan
Copy link

🤖 AI Code Review

Risk Level: 🟢 LOW

Summary

README updated from a simple greeting to include example Git commands for setting up a repository. The changes add instructional content but introduce formatting issues and potential security concerns.

Key Changes

  • README content expanded from 'Hello World!' to include multiple lines of example shell commands and descriptions.
  • Added commands: mkdir, cd, git init, touch.
  • No newline at end of file remains.

Findings

🟢 LOW - security
📁 README

Lines added include shell commands with user directory paths (e.g., '/Users/your_user_directory/Hello-World/.git/').

💡 Consider using placeholders or generic examples without absolute paths to avoid potential information disclosure or confusion.

🟡 MEDIUM - style
📁 README

Added lines mix command prompts ($) with descriptions without clear formatting (e.g., 'Creates a directory...' appended directly after command).

💡 Format commands and descriptions separately for better readability, such as using code blocks or bullet points.

🟢 LOW - correctness
📁 README

The diff shows 'No newline at end of file' for both old and new versions, which may cause issues in some tools.

💡 Ensure the file ends with a newline character to adhere to POSIX standards.

Suggested Tests

  • Verify README renders correctly in markdown viewers.
  • Check that any automated scripts processing README handle the new content without errors.

🤖 Reviewed by AI Code Reviewer | Model: deepseek/deepseek-chat | Developed by Vardhan Jalluri

@kdehl16-web
Copy link

Lots of comments bro 💀 ☠️

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

@kdehl16-web
Copy link

why are they hidden items
Screenshot 2026-02-20 110421

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

@kdehl16-web
Copy link

Remove more hidden items

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

Hello World! No newline at end of file
Hello World!
$ mkdir ~/Hello-WorldCreates a directory for your project called "Hello-World" in your user directory
$ cd ~/Hello-WorldChanges the current working directory to your newly created directory
Copy link

@composio-partner-123 composio-partner-123 Feb 23, 2026

Choose a reason for hiding this comment

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

Updated comment on octocat repo: Testing minimal body text.

Choose a reason for hiding this comment

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

I agree with this feedback! Let's make those changes.

$ mkdir ~/Hello-WorldCreates a directory for your project called "Hello-World" in your user directory
$ cd ~/Hello-WorldChanges the current working directory to your newly created directory
$ git initSets up the necessary Git files
Initialized empty Git repository in /Users/your_user_directory/Hello-World/.git/
Copy link

@composio-partner-123 composio-partner-123 Feb 23, 2026

Choose a reason for hiding this comment

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

Extended test with longer content:

This is a comprehensive review comment that includes multiple paragraphs and various formatting elements to test the robustness of the update action.

Section 1: Code Quality

The code looks good overall, but there are a few areas that could be improved:

  • Better error handling
  • More comprehensive tests
  • Documentation updates

Section 2: Performance

Consider optimizing the following:

  1. Database queries could be batched
  2. Caching strategy needs review
  3. Memory usage seems high

Section 3: Security

Please address these security concerns:

  • Input validation
  • Authentication checks
  • Authorization rules

Let me know if you have any questions! 🚀

Comment on lines +2 to +4
$ mkdir ~/Hello-WorldCreates a directory for your project called "Hello-World" in your user directory
$ cd ~/Hello-WorldChanges the current working directory to your newly created directory
$ git initSets up the necessary Git files

Choose a reason for hiding this comment

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

This section could be improved with better formatting.

@@ -1 +1,6 @@
Hello World! No newline at end of file

Choose a reason for hiding this comment

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

Testing with deprecated position parameter.

Copy link

@composio-partner-123 composio-partner-123 left a comment

Choose a reason for hiding this comment

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

This is a test review comment for testing purposes.

@neo-noclick
Copy link

Test comment from automated tests.

Copy link

@neo-noclick neo-noclick left a comment

Choose a reason for hiding this comment

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

Test review comment

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.