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

[Documentation Update] Update submission guide & Building instructions #1003

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

EnterpriseGoose
Copy link
Contributor

Summary of changes:

Building instructions

  • Updated a few steps that were missing or mistyped
  • Added a couple debugging tips

Submission guidelines

  • Specified that not only color but the entire fill option is not drawable

PR template

  • Updated to match submission guidelines

Add more details on what constitutes too much overlap and how the entire fill option is not supported (not just colored fills).
Made various changes:
- Some steps I found when I was building the blot were mistyped or missing
- Added a couple debugging tips so people can learn from my failures
Specify that it's not only color, but also the fill option as a whole that doesn't work on blot.
Copy link

vercel bot commented Nov 21, 2024

@EnterpriseGoose is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Member

@krishnans2006 krishnans2006 left a comment

Choose a reason for hiding this comment

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

Not a reviewer, but I fully agree with these changes :)

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.

2 participants