Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add `decorates_assigned` to README #511

Merged
merged 2 commits into from Apr 4, 2013

Conversation

Projects
None yet
2 participants
Contributor

haines commented Apr 4, 2013

No description provided.

Owner

steveklabnik commented Apr 4, 2013

80 char wrap and then let's commit! :D

80-char wrap
[ci skip]
Contributor

haines commented Apr 4, 2013

Done :)

steveklabnik added a commit that referenced this pull request Apr 4, 2013

@steveklabnik steveklabnik merged commit 18e27a5 into drapergem:master Apr 4, 2013

Owner

steveklabnik commented Apr 4, 2013

👍

Owner

steveklabnik commented Apr 4, 2013

Btw I'm getting emails for your fork of draper.

@haines haines deleted the haines:decorates_assigned_readme branch Apr 4, 2013

Contributor

haines commented Apr 4, 2013

Weird... did you just get them for commits to this branch (decorates_assigned_readme), or are you getting them for other branches? Possibly a feature, if just for this branch, in order to notify you of updates to the PR. Otherwise definitely a bug, you aren't listed as a collaborator or watcher anywhere on my fork (that I can see).

Owner

steveklabnik commented Apr 4, 2013

I just got a failure notification that your build for this pull error-ed. said it was for haines/draper

Contributor

haines commented Apr 4, 2013

Interesting. I think Travis does something where the build status for a PR can be determined from the build on the forked repo, at least until a build is run on the merge commit. Maybe that's why you got it? Odd, anyway.

Owner

steveklabnik commented Apr 4, 2013

It's no big deal, if it was something obvious and often, we could figure it out, but I just figured i'd mention it.

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