Skip to content

Add my fantasy cow#14

Closed
jujusjuicebox wants to merge 2 commits into
codeday:mainfrom
jujusjuicebox:add-luna-cow
Closed

Add my fantasy cow#14
jujusjuicebox wants to merge 2 commits into
codeday:mainfrom
jujusjuicebox:add-luna-cow

Conversation

@jujusjuicebox

Copy link
Copy Markdown

my fantasy cow added

@github-actions

Copy link
Copy Markdown

🧪 PR Validation Results for #14


❌ Multiple JSON files found! Please include only one .json file.

@cla-assistant

cla-assistant Bot commented Apr 13, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions

Copy link
Copy Markdown

🧪 PR Validation Results for #14


❌ Multiple JSON files found! Please include only one .json file.

1 similar comment
@github-actions

Copy link
Copy Markdown

🧪 PR Validation Results for #14


❌ Multiple JSON files found! Please include only one .json file.

@github-actions

Copy link
Copy Markdown

🧪 PR Validation Results for #14

Check PR is going to correct repo passed!
Check image path passed!
❌ Image file images/example_cow.jpeg specified in cows/ajohn.json does not exist in the PR.
❌ Image file is too large! Maximum allowed size is 200 KB. Please resize or compress it and update your PR.
❌ Image file name in cows/ajohn.json should be based on the cow's name. Expected: images/luna.png or .jpg or .jpeg
❌ File cows/ajohn.json is not properly indented.
Check line endings passed!


❌ 3/7 passed. Please fix the issues above and update your PR.

⚠️ You have 3 previous failed validation attempts. Please schedule a CSE session using your student dashboard.

@github-actions

Copy link
Copy Markdown

🧪 PR Validation Results for #14


❌ Multiple JSON files found! Please include only one .json file.

⚠️ You have 4 previous failed validation attempts. Please schedule a CSE session using your student dashboard.

@oohwooh

oohwooh commented Jul 2, 2026

Copy link
Copy Markdown
Member

Hi @jujusjuicebox - thanks so much for your contribution, I love your cow!

We'd love to merge your PR, can you please fix the issues identified?

Please feel free to reach out if you have any questions or need help :)

@oohwooh

oohwooh commented Jul 18, 2026

Copy link
Copy Markdown
Member

Hi there,

I'm closing this due to inactivity, if you're still interested in contributing a cow feel free to open a new PR!

It's really common in open source to need to update your PR to meet maintainer requests or pass automated checks. One of the reasons we made this project is to help new contributors get used to receiving and addressing maintainer feedback, which happens every day in open source!

You're welcome to reach out any time for help or to ask questions :)

@oohwooh oohwooh closed this Jul 18, 2026
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