Skip to content

New feature#265

Open
CamiloVelasco2 wants to merge 2 commits intoRahafB:mainfrom
CamiloVelasco2:new-feature
Open

New feature#265
CamiloVelasco2 wants to merge 2 commits intoRahafB:mainfrom
CamiloVelasco2:new-feature

Conversation

@CamiloVelasco2
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings April 9, 2026 00:34
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a small greeting script and modifies an existing greeter script to print an additional line of output.

Changes:

  • Added camilo_script.sh that echoes a simple greeting.
  • Updated 10-greeter.sh to print an extra informational line after reading the user’s name.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
camilo_script.sh New bash script that prints a greeting.
10-greeter.sh Adds an extra echo line to the interactive greeter flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

echo "Please enter your name"
read name

echo "This line was added by Camilo on the new-feature branch"
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

The added echo prints branch/author metadata every run, which changes the script’s user-facing output without affecting greeting logic. Please remove it, or gate it behind an explicit verbose/debug flag if it’s intended for diagnostics.

Copilot uses AI. Check for mistakes.
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.

2 participants