Skip to content

Conversation

@SThor
Copy link
Owner

@SThor SThor commented Apr 18, 2025

No description provided.

@SThor SThor requested a review from Copilot April 27, 2025 21:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new "Grotto" sketch by adding a related entry in the todo list and creating a new readme for the grotto.

  • Updates the todo list with a new task for experimenting with dark shadows in the drawing process
  • Adds a basic readme for the grotto feature

Reviewed Changes

Copilot reviewed 20 out of 21 changed files in this pull request and generated 1 comment.

File Description
todo.md Adds a new task under grotto and removes duplicate Color sub-tasks
grotto/readme.md Provides a basic readme introducing the Grotto sketch feature
Files not reviewed (1)
  • grotto/grotto.pde: Language not supported

- [ ] colorGrowth:
- [ ] Try it with a more complex color system
- [ ] grotto
- [ ] try reversing the drawing process, creating dark shadows instead of light ridges
Copy link

Copilot AI Apr 27, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider capitalizing the first word for consistency with other task entries, e.g., 'Try reversing the drawing process...'.

Suggested change
- [ ] try reversing the drawing process, creating dark shadows instead of light ridges
- [ ] Try reversing the drawing process, creating dark shadows instead of light ridges

Copilot uses AI. Check for mistakes.
Copy link
Owner Author

Choose a reason for hiding this comment

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

Nope it's fine, this file doesn't need to be perfect, it's simply a quick todolist

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.

1 participant