Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

fix(app) change the script src to lowercase #558

Closed
wants to merge 1 commit into from
Closed

fix(app) change the script src to lowercase #558

wants to merge 1 commit into from

Conversation

zhuangya
Copy link
Contributor

No description provided.

@passy
Copy link
Member

passy commented Jan 27, 2014

@zhuangya
Copy link
Contributor Author

@passy hey, thanks, but do you mind have a look at #557 ?

@zhuangya
Copy link
Contributor Author

ahh, sorry, let me have a look at travis, sorry for bothering.

@passy
Copy link
Member

passy commented Jan 27, 2014

@zhuangya Tests should be fine, @eddiemonge is looking into that at the moment. I was referring to the commit message.

@eddiemonge
Copy link
Member

the commit message doesnt follow the guidelines. although the code this is fixing won't matter in about a week but thank you. if you fix the message we will still land this fix.

unify the scriptsrc and filename

fix #557
@zhuangya
Copy link
Contributor Author

does it look better now?

@eddiemonge

@eddiemonge
Copy link
Member

close enough. i modified it slightly

@eddiemonge eddiemonge closed this Jan 27, 2014
@zhuangya
Copy link
Contributor Author

@eddiemonge thank you very much 👍

@zhuangya zhuangya deleted the case_sensitive branch January 27, 2014 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants