Skip to content
This repository has been archived by the owner on Nov 27, 2022. It is now read-only.

misc/bot is too old #205

Merged
merged 1 commit into from
Nov 24, 2014
Merged

misc/bot is too old #205

merged 1 commit into from
Nov 24, 2014

Conversation

mallowlabs
Copy link
Member

misc/bot is useful as a sample code.
But it has two problems.

  • & in message body will be broken
  • simple.rb is duplicated to simple-ssl.rb. There should be unified

@suer
Copy link
Member

suer commented Nov 24, 2014

LGTM

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling cf39813 on mallowlabs:id/205 into 961c16c on codefirst:master.

mallowlabs added a commit that referenced this pull request Nov 24, 2014
@mallowlabs
Merge pull request #205 from mallowlabs/id/205
dea22b9 
misc/bot is too old
@mallowlabs mallowlabs merged commit dea22b9 into codefirst:master Nov 24, 2014
@mallowlabs mallowlabs deleted the id/205 branch November 24, 2014 04:29
@mallowlabs
Copy link
Member Author

Thanks 😸

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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mallowlabs @suer @coveralls