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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 30880

View Options

src/workflow/ArcanistWorkflow.php

Fix missing comma in sprintf() in ArcanistWorkflow.ClosedPublicActions