Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update comment #37

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Update comment #37

wants to merge 2 commits into from

Conversation

TristanBrotherton
Copy link
Owner

@TristanBrotherton TristanBrotherton commented Feb 1, 2023

Test PR


This change is Reviewable

Copy link
Owner Author

@TristanBrotherton TristanBrotherton left a comment

Choose a reason for hiding this comment

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

I added one comment

## https://github.com/TristanBrotherton/netcheck ##
## -- Tristan Brotherton ##
################################################################################

Copy link
Owner Author

Choose a reason for hiding this comment

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

This was amazing

Copy link
Owner Author

Choose a reason for hiding this comment

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

Another comment

Copy link
Owner Author

Choose a reason for hiding this comment

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

I ate breakfast

Choose a reason for hiding this comment

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

I really like this

Copy link
Owner Author

Choose a reason for hiding this comment

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

Can you add a some code here to define a new function

Suggested change
#!/bin/bash
check_connection() {
# Function body will be implemented later
echo "Checking connection..."
}

@TristanBrotherton
Copy link
Owner Author

netcheck.sh line 3 at r1 (raw file):

Previously, TristanBrotherton (Tristan Brotherton) wrote…

This was amazing

This is a follow up

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