Skip to content

Conversation

@SanjnaT41756
Copy link
Contributor

Change "Things You should do" section

Summary

Closes #

Test Plan

SanjnaT41756 and others added 3 commits September 10, 2024 13:06
@SanjnaT41756 SanjnaT41756 requested a review from neilwj7 March 5, 2025 02:58
@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for pen-pals ready!

Name Link
🔨 Latest commit 0a9b4a4
🔍 Latest deploy log https://app.netlify.com/sites/pen-pals/deploys/67cf4fb19505bf0008f44c05
😎 Deploy Preview https://deploy-preview-166--pen-pals.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@neilwj7 neilwj7 left a comment

Choose a reason for hiding this comment

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

Hi Sanjna! Would you be able to 1. run the lint fix command (its in the readme) to run the linter on your code and 2. merge the main branch into your branch so your branch has the updated main code?

Let me know if you have any questions!

@neilwj7
Copy link
Collaborator

neilwj7 commented Mar 10, 2025

Looks good!

@neilwj7 neilwj7 closed this Mar 10, 2025
@SanjnaT41756 SanjnaT41756 reopened this Mar 10, 2025
@SanjnaT41756 SanjnaT41756 merged commit d3205a7 into main Mar 10, 2025
6 checks passed
@SanjnaT41756 SanjnaT41756 deleted the SanjnaT41756-patch-1 branch March 10, 2025 23:13
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