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

feat(act 1): activity 1 - clean code basics #11

Merged
merged 10 commits into from
Oct 11, 2024

Conversation

lwasser
Copy link
Member

@lwasser lwasser commented Oct 7, 2024

This is a single PR that adds the first activity to our lessons. It is ready for review if anyone would like to have a look.


* provide it with descriptive, leading prompts that allow it to perform the task better. So you might write:

> Make the variables in the code below.
Copy link
Contributor

Choose a reason for hiding this comment

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

Incomplete sentence?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed! thanks!

Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

Looking good. Though I really wanted to edit the code snippet. lol

clean-modular-code/activity-1/clean-code-activity-1.md Outdated Show resolved Hide resolved
clean-modular-code/activity-1/clean-code-activity-1.md Outdated Show resolved Hide resolved
clean-modular-code/activity-1/clean-code-activity-1.md Outdated Show resolved Hide resolved
clean-modular-code/activity-1/clean-code-activity-1.md Outdated Show resolved Hide resolved
@lwasser
Copy link
Member Author

lwasser commented Oct 7, 2024

Looking good. Though I really wanted to edit the code snippet. lol

@willingc 😆 do you mean the code snippet that is the basis for the activity? 🙃 if so - i know - it is really hard to publish something like that now.

@lwasser
Copy link
Member Author

lwasser commented Oct 11, 2024

@all-contributors please add @ucodery for review

Copy link
Contributor

@lwasser

I've put up a pull request to add @ucodery! 🎉

@lwasser
Copy link
Member Author

lwasser commented Oct 11, 2024

Ok friends - thank you for the reviews!! There will be one htmlproofer error here that will be fixed in #13 All of the other suggestions and edits have been implemented! 🚀 i will merge this when ci is complete!

@lwasser lwasser merged commit 54cc454 into pyOpenSci:main Oct 11, 2024
1 of 2 checks passed
@lwasser lwasser deleted the activity-1 branch October 11, 2024 16:30
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.

3 participants