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

Add work#87

Open
dillon-sellars wants to merge 2 commits into
masterfrom
feature/fix_problem
Open

Add work#87
dillon-sellars wants to merge 2 commits into
masterfrom
feature/fix_problem

Conversation

@dillon-sellars

Copy link
Copy Markdown

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
Copy Markdown

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.

@dsellarsnr dsellarsnr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Introduce typo

Comment thread work.py
import argparse

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

Copy link
Copy Markdown

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

@dsellarsnr dsellarsnr left a comment

Copy link
Copy Markdown

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

@dsellarsnr dsellarsnr left a comment

Copy link
Copy Markdown

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
Copy Markdown

one last comment

Comment thread work.py
@@ -2,7 +2,7 @@

import argparse

Copy link
Copy Markdown

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