Page MenuHomePhabricator

made email commands extraction a bit more lax
Needs RevisionPublic

Authored by talshiri on Jun 6 2014, 2:04 AM.
Tags
None
Referenced Files
F12835992: D9405.diff
Thu, Mar 28, 3:31 PM
F12827280: D9405.id22446.diff
Thu, Mar 28, 9:54 AM
Unknown Object (File)
Thu, Mar 7, 2:44 PM
Unknown Object (File)
Feb 4 2024, 3:59 AM
Unknown Object (File)
Dec 27 2023, 10:50 PM
Unknown Object (File)
Dec 25 2023, 1:16 AM
Unknown Object (File)
Dec 13 2023, 7:32 PM
Unknown Object (File)
Dec 13 2023, 4:57 PM

Details

Summary

allows !command to also appear at the end, and be less sensitive about whitespaces

Test Plan

wrote unit tests.

Diff Detail

Repository
rP Phabricator
Branch
friendlier_reply_commands
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 894
Build 894: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

talshiri retitled this revision from to made email commands extraction a bit more lax.
talshiri updated this object.
talshiri edited the test plan for this revision. (Show Details)
talshiri added a reviewer: epriestley.

ping? There's totally tests for it :D

I wanted to go in a slightly different direction with this, here's the half-a-diff I have locally that I haven't had a chance to finish/test yet:

https://secure.phabricator.com/differential/diff/23440/

In particular, the rules are:

  • !command is parsed on any line, but must appear at the beginning of the line.
  • Multiple !commands are allowed to appear in a message (e.g., !assign + !close).
  • Words after !command are interpreted as arguments.

In particular, I don't think "just chatting about !reject" should trigger a reject. The other stuff makes this more flexible in the future.

epriestley edited edge metadata.

This is obsoleted by changes connected to T7199.

This revision now requires changes to proceed.May 17 2015, 1:32 PM