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

Added a baseto the drawing #7

Merged
merged 3 commits into from Oct 7, 2023

Conversation

Satoshi-Sh
Copy link
Contributor

Issue number

#3
[Your PR must reference an issue. You can reference the issue by using the # character]

Description

Added a function to draw a base for the Christmas tree.
[Please describe your contribution]

Checklist

  • [ x] I have read and complied with the Code of Conduct and Content Guidelines as appropriate.
  • [ x] I have followed the Contributing Guide.
  • [x ] I agree to license my contribution under the License in this repository.
  • [ x] The program runs successfully after making my changes.

Copy link
Owner

@dominicduffin1 dominicduffin1 left a comment

Choose a reason for hiding this comment

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

Thank you @Satoshi-Sh! Looks good, just a couple of minor suggestions!

drawfunctions/satoshi_s.py Outdated Show resolved Hide resolved
drawfunctions/satoshi_s.py Show resolved Hide resolved
@dominicduffin1
Copy link
Owner

Thank you @Satoshi-Sh, merging now!

@dominicduffin1 dominicduffin1 merged commit 8d36082 into dominicduffin1:parent Oct 7, 2023
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.

None yet

2 participants