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

First Contribution: I added a circle to the confetti, I think #672

Merged
merged 6 commits into from
Nov 7, 2022

Conversation

JHMuir
Copy link
Collaborator

@JHMuir JHMuir commented Nov 4, 2022

No description provided.

@vercel
Copy link

vercel bot commented Nov 4, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @EthanThatOneKid on Vercel.

@EthanThatOneKid first needs to authorize it.

@JHMuir JHMuir changed the title I added a circle to the confetti, I think First Contribution: I added a circle to the confetti, I think Nov 4, 2022
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

There is only a minor change keeping you from saying, "first try 😎", but luckily it is a super minor change needed to make your SVG file valid. I left a suggestion in my review, feel free to commit the suggestion directly or add it manually if you have additional changes :).

static/assets/1st/circle-thing.svg Outdated Show resolved Hide resolved
@jaasonw
Copy link
Contributor

jaasonw commented Nov 6, 2022

Also for future reference, your pull requests should contain the following
image

Renamed 'my-new-thing.ts' to 'circle-thing.ts'

Fixed the dimensions of 'circle-thing.svg'
@vercel
Copy link

vercel bot commented Nov 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
acm-csuf-site ✅ Ready (Inspect) Visit Preview Nov 7, 2022 at 11:07PM (UTC)

@vercel vercel bot temporarily deployed to Preview November 7, 2022 23:07 Inactive
Copy link
Owner

@EthanThatOneKid EthanThatOneKid left a comment

Choose a reason for hiding this comment

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

Nice first contribution!

@EthanThatOneKid EthanThatOneKid merged commit d76c03c into EthanThatOneKid:main Nov 7, 2022
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