-
Notifications
You must be signed in to change notification settings - Fork 8
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
new pull request to add michelange #273
Conversation
Visit the preview URL for this PR (updated for commit f833263): https://ccv-brown-edu--pr273-michelange-senat-1my1uohj.web.app (expires Tue, 25 Jun 2024 15:12:19 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: e6225babbaf68dadb99b2906d33811740e5d15cb |
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.
The format of the yaml looks good and the images too! I made some comments on typos and work changes. Looks like you've replaced Jesus! So let's bring him back into the website.
Looks good so far though !
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.
added Jesus back and fixed my errors
Looks great after your revisions! One little typo and I think we're good to merge! |
You'll have to merge main in and resolve the conflicts. Let me know if you'd like help with that |
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.
Looks great, thank you! 🎉
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.
done!
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.
Amazing!!! Merge it in when you get the chance :)
No description provided.