Skip to content
This repository was archived by the owner on Dec 27, 2023. It is now read-only.

Fix 311 lint errors #23

Open
sevazhidkov wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix 311 lint errors #23

sevazhidkov wants to merge 1 commit into from

Conversation

@sevazhidkov
Copy link

@sevazhidkov sevazhidkov commented Jan 23, 2016

No description provided.

@smcgregor
Copy link
Member

smcgregor commented Feb 26, 2016

Hi @sevazhidkov! A few comments:

  • We use double quotes for the outer quotation. We are formalizing this by adopting the jQuery coding standard.
  • You should make all pull requests against the develop branch, if it exists for the repository in question.
  • Please summarize your pull requests. In this case, it would be a list of the type of lint errors you are fixing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

Needs Work

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sevazhidkov @smcgregor