Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

Changed 1 file … #73

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Changed 1 file … #73

wants to merge 5 commits into from

Conversation

marcelofarias
Copy link
Contributor

What does this PR do?

N/A

Description of Task to be completed?

N/A

Where should the reviewer start?

Place link to file here

How should this be manually tested?

N/A

Any background context you want to provide?

N/A

What is the relevant trello card?

Place Trello id here

Screenshots (if appropriate)

N/A

Created from JetBrains using CodeStream

@dsellarsnr dsellarsnr assigned dsellarsnr and unassigned dsellarsnr Mar 21, 2022
@dsellarsnr dsellarsnr requested review from dsellarsnr and removed request for dsellarsnr March 21, 2022 21:23
@dsellarsnr dsellarsnr assigned dsellarsnr and unassigned dsellarsnr Mar 21, 2022
@dsellarsnr dsellarsnr self-requested a review April 1, 2022 17:07
Comment on lines +7 to 8
// also changed this file
my $html = qx{curl --silent $url};
Copy link
Contributor Author

Choose a reason for hiding this comment

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

foo

Comment on lines 14 to 16
foreach (@ARGV) {
// changed this file
find_emails($_);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

allalalalala

@@ -12,6 +12,7 @@ sub find_emails {

# get all emails from command-line args
foreach (@ARGV) {
// changed this file
Copy link
Contributor Author

Choose a reason for hiding this comment

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

lelelelelelel

Comment on lines +15 to 17
// changed this file
find_emails($_);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

a review

Copy link
Contributor Author

@marcelofarias marcelofarias left a comment

Choose a reason for hiding this comment

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

asdf

Comment on lines 14 to 17
foreach (@ARGV) {
// changed this file
find_emails($_);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

asdfsad

Comment on lines +15 to 17
// changed this file
find_emails($_);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

aaaaa

Comment on lines +11 to +12
// changed this file again one more time, over and over
$emails_found{$1}++;

Choose a reason for hiding this comment

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

Perfect

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.

3 participants