Removed from note.spec.js unnecessary # on id #25

Merged
merged 1 commit into from Mar 19, 2015

Conversation

Projects
None yet
3 participants
@diegocasmo
Contributor

diegocasmo commented Mar 17, 2015

No description provided.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Mar 17, 2015

Coverage Status

Coverage remained the same at 83.13% when pulling fa4d967 on diegocasmo:master into 106eb7f on ryan-roemer:master.

Coverage Status

Coverage remained the same at 83.13% when pulling fa4d967 on diegocasmo:master into 106eb7f on ryan-roemer:master.

@ryan-roemer

This comment has been minimized.

Show comment
Hide comment
@ryan-roemer

ryan-roemer Mar 19, 2015

Owner

@diegocasmo -- Thanks! The CI setup is somehow messed up and I'm trying to fix that first before merging, but I'll get your change in one way or another...

Owner

ryan-roemer commented Mar 19, 2015

@diegocasmo -- Thanks! The CI setup is somehow messed up and I'm trying to fix that first before merging, but I'll get your change in one way or another...

@ryan-roemer

This comment has been minimized.

Show comment
Hide comment
@ryan-roemer

ryan-roemer Mar 19, 2015

Owner

CI looks fine on master, so I'm just going to merge this and hope for the best once it lands. ;)

Thanks again @diegocasmo !

Owner

ryan-roemer commented Mar 19, 2015

CI looks fine on master, so I'm just going to merge this and hope for the best once it lands. ;)

Thanks again @diegocasmo !

ryan-roemer added a commit that referenced this pull request Mar 19, 2015

Merge pull request #25 from diegocasmo/master
Removed from note.spec.js unnecessary # on id

@ryan-roemer ryan-roemer merged commit bd43fec into ryan-roemer:master Mar 19, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment