Paths

Table of Contentst

Differential D12837

Fix missing comma in sprintf() in ArcanistWorkflow.
ClosedPublic
Actions

Authored by nevogd on May 14 2015, 12:59 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8195: Error running arc diff command: sprintf(): Too few arguments at ArcanistWorkflow.php:1024

Commits

rARC383633e63fd7: Fix missing comma in sprintf() in ArcanistWorkflow.

Summary

Refs D12607. Fixes T8195. Replace period in the sprintf() arguments with a comma.

Test Plan

Ran arc diff in the patch applied, did not get the sprintf() error

Diff Detail

Repository

rARC Arcanist

Branch

workflow_sprintf

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5975
Build 5995: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

nevogd updated this revision to Diff 30879.May 14 2015, 12:59 PM

nevogd retitled this revision from to Fix missing comma in sprintf() in ArcanistWorkflow..

nevogd updated this object.

nevogd edited the test plan for this revision. (Show Details)

nevogd added a task: T8195: Error running arc diff command: sprintf(): Too few arguments at ArcanistWorkflow.php:1024.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 14 2015, 12:59 PM

Herald added a subscriber: epriestley. · View Herald Transcript

epriestley accepted this revision.May 14 2015, 12:59 PM

epriestley added a reviewer: epriestley.

This revision is now accepted and ready to land.May 14 2015, 12:59 PM

Closed by commit rARC383633e63fd7: Fix missing comma in sprintf() in ArcanistWorkflow. (authored by NeverOutgunned, committed by epriestley). · Explain WhyMay 14 2015, 12:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistWorkflow.php

2 lines

Diff 30879

View Options

src/workflow/ArcanistWorkflow.php

Fix missing comma in sprintf() in ArcanistWorkflow.ClosedPublicActions