Skip to content

Commit

Permalink
Merge pull request #1408 from hydephp/code-cleanup
Browse files Browse the repository at this point in the history
Remove low priority todo comments hydephp/develop@d5b24d8
  • Loading branch information
github-actions committed Oct 28, 2023
1 parent d6bac3a commit 5a9ea3e
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/Framework/Features/Blogging/Models/FeaturedImage.php
Original file line number Diff line number Diff line change
Expand Up @@ -225,9 +225,7 @@ protected function getContentLengthForLocalImage(): int

protected function getContentLengthForRemoteImage(): int
{
// TODO: We may want to globalize this check in the config, but for now,
// we just check the server arguments and skip remote requests if
// the --no-api flag is present (in the build command call)
// Check if the --no-api flag is set when running the build command, and if so, skip the API call.
if (! (isset($_SERVER['argv']) && in_array('--no-api', $_SERVER['argv'], true))) {
$headers = Http::withHeaders([
'User-Agent' => Config::getString('hyde.http_user_agent', 'RSS Request Client'),
Expand Down

0 comments on commit 5a9ea3e

Please sign in to comment.