-
-
Notifications
You must be signed in to change notification settings - Fork 99
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
fix: Created LEARN.md #335
Conversation
@Meetjain1 is attempting to deploy a commit to the Sanchit Bajaj's projects Team on Vercel. A member of the Team first needs to authorize it. |
Warning Rate Limit Exceeded@Meetjain1 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 42 minutes and 38 seconds before requesting another review. How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. WalkthroughThe Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, @Meetjain1! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project. Keep being the part of the community!
'We will promptly review your changes and offer feedback. Keep up the excellent work!
Kindly remember to check our Contributing Guidelines'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 4
Outside diff range and nitpick comments (1)
LEARN.md (1)
47-47
: Consider simplifying the language for clarity.The phrase "Individuals with expertise in" could be simplified to "Experts in" to make the sentence more concise and direct.
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- LEARN.md (1 hunks)
Additional Context Used
LanguageTool (1)
LEARN.md (1)
Near line 47: ‘In’ is unnecessary in most cases before the expression ‘next .’.
Context: ...his project? Individuals with expertise in Next.js, TypeScript, Tailwind CSS, and Appwr...
Rule ID: CONFUSION_OF_NN_IN_NEXT_NNP
Markdownlint (21)
LEARN.md (21)
25: Expected: 2; Actual: 3
Unordered list indentation
29: Expected: 2; Actual: 3
Unordered list indentation
33: Expected: 2; Actual: 3
Unordered list indentation
37: Expected: 2; Actual: 3
Unordered list indentation
38: Expected: 0 or 2; Actual: 1
Trailing spaces
23: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
27: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
31: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
35: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
43: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
46: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
49: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
52: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
55: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
58: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
61: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
64: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
24: null
Lists should be surrounded by blank lines
28: null
Lists should be surrounded by blank lines
32: null
Lists should be surrounded by blank lines
36: null
Lists should be surrounded by blank lines
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 4
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- LEARN.md (1 hunks)
Additional Context Used
LanguageTool (1)
LEARN.md (1)
Near line 46: ‘In’ is unnecessary in most cases before the expression ‘next .’.
Context: ...his project? Individuals with expertise in Next.js, TypeScript, Tailwind CSS, and Appwr...
Rule ID: CONFUSION_OF_NN_IN_NEXT_NNP
Markdownlint (21)
LEARN.md (21)
25: Expected: 2; Actual: 3
Unordered list indentation
29: Expected: 2; Actual: 3
Unordered list indentation
33: Expected: 2; Actual: 3
Unordered list indentation
37: Expected: 2; Actual: 3
Unordered list indentation
23: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
27: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
31: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
35: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
42: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
45: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
48: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
51: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
54: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
57: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
60: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
63: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines
24: null
Lists should be surrounded by blank lines
28: null
Lists should be surrounded by blank lines
32: null
Lists should be surrounded by blank lines
36: null
Lists should be surrounded by blank lines
37: null
Lists should be surrounded by blank lines
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@Sanchitbajaj02 have a look on this pr . Thankyou |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@Sanchitbajaj02 gssoc label is wrong please change it . |
Thanks, for informing. It is changed now |
Related Issue
closes #334
Description
created LEARN.md file from scratch.
Summary by CodeRabbit
LEARN.md
file to introduce the Palettegram Website project, tech stack, learning resources, and FAQ section.