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

Add work #87

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

Add work #87

wants to merge 2 commits into from

Conversation

dillon-sellars
Copy link

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

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

@dsellarsnr dsellarsnr left a comment

Choose a reason for hiding this comment

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

Introduce typo

@@ -2,7 +2,7 @@

import argparse

parser = argparse.ArgumentParser(description='Do some work')
parser = argparse.ArgumentParser(description='Do some moar work')

Choose a reason for hiding this comment

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

Nice

dsellarsnr
dsellarsnr previously approved these changes Aug 29, 2022
Copy link

@dsellarsnr dsellarsnr left a comment

Choose a reason for hiding this comment

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

Good job

@dsellarsnr dsellarsnr dismissed their stale review August 29, 2022 23:30

Oops not approved

dsellarsnr
dsellarsnr previously approved these changes Aug 29, 2022
Copy link

@dsellarsnr dsellarsnr left a comment

Choose a reason for hiding this comment

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

All better

@dsellarsnr
Copy link

one last comment

@@ -2,7 +2,7 @@

import argparse

Choose a reason for hiding this comment

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

Good import

dsellarsnr
dsellarsnr previously approved these changes Aug 30, 2022
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