Skip to content

Commit

Permalink
Clean up some todos.
Browse files Browse the repository at this point in the history
  • Loading branch information
bobthecow committed Dec 6, 2023
1 parent 6b4c749 commit 7a3950b
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 6 deletions.
2 changes: 0 additions & 2 deletions src/ExecutionLoop/RunkitReloader.php
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ public static function isSupported(): bool

/**
* Construct a Runkit Reloader.
*
* @todo Pass in Parser Factory instance for dependency injection?
*/
public function __construct()
{
Expand Down
3 changes: 1 addition & 2 deletions src/ExecutionLoopClosure.php
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@
/**
* The Psy Shell's execution loop scope.
*
* @todo Once we're on PHP 5.5, we can switch ExecutionClosure to a generator
* and get rid of the duplicate closure implementations :)
* @todo Switch ExecutionClosure to a generator and get rid of the duplicate closure implementations?
*/
class ExecutionLoopClosure extends ExecutionClosure
{
Expand Down
2 changes: 1 addition & 1 deletion src/Readline/Hoa/File.php
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ public function newBuffer($callable = null, int $size = null): int
{
$this->setStreamBuffer($size);

// @TODO manage $callable as a filter?
// @todo manage $callable as a filter?
return 1;
}
Expand Down
2 changes: 1 addition & 1 deletion src/functions.php
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,7 @@ function info(Configuration $config = null)
];

$theme = $config->theme();
// TODO: show styles (but only if they're different than default?)
// @todo show styles (but only if they're different than default?)
$output['theme'] = [
'compact' => $theme->compact(),
'prompt' => $theme->prompt(),
Expand Down

0 comments on commit 7a3950b

Please sign in to comment.