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

Homework Week2 #13

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from
Open

Homework Week2 #13

wants to merge 2 commits into from

Conversation

CleezyITP
Copy link

No description provided.

@afeld
Copy link
Member

afeld commented Sep 23, 2014

Hey Claire-
Just to respond to your question via email:

It says "build failed" and there is a big red "x" next to my name. What does that mean?

There's general information in the workflow guide, but you can see the specific errors by clicking the details link. I'll leave inline comments to clarify what they mean.

};


function Mytimer(){
Copy link
Member

Choose a reason for hiding this comment

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

This error is referring to the convention that SentenceCase function names in JavaScript are a convention for a "class", which this is not. It was expecting new when you called it within the $(document).ready(), because that's how constructor functions get used. If you make the function all lower or camelCase, the error will go away.

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