Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

EC-5234 ignore mkdir warnings #92

Merged
merged 2 commits into from Dec 4, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/Builder/BuildFiles.php
Expand Up @@ -184,7 +184,7 @@ private function checkIfAnyChanged(File $output_file, int $mtime, array $input_f
if (!file_exists($sources_file)) {
// make sure the cache dir exists
if (!is_dir(\dirname($sources_file))) {
mkdir(\dirname($sources_file), 0777, true);
@mkdir(\dirname($sources_file), 0777, true);
}
file_put_contents($sources_file, serialize($input_sources));

Expand Down
2 changes: 1 addition & 1 deletion src/Builder/Bundler.php
Expand Up @@ -58,7 +58,7 @@ public function bundle(BuildConfig $build_config): void
'--debug',
'--log-json',
'--stdin',
$config_file
$config_file,
];

$process = new Process($cmd, $this->config->getProjectRoot(), [
Expand Down
2 changes: 1 addition & 1 deletion src/Import/Nodejs/FileResolver.php
Expand Up @@ -122,7 +122,7 @@ private function asIndex(string $name): string
// 1. If X/index.js is a file, load X/index.js as JavaScript text. STOP
foreach ($this->extensions as $ext) {
if (is_file($path . '/index' . $ext)) {
return File::clean($name . '/index'. $ext);
return File::clean($name . '/index' . $ext);
}
}

Expand Down
2 changes: 1 addition & 1 deletion test/Import/BuiltIn/TsImportCollectorTest.php
Expand Up @@ -84,7 +84,7 @@ public function testCollect()
new Import('jquery', new Module('jquery', 'node_modules/jquery/jquery.js')),
new Import('./Alias', new File('resolver/ts/import-syntax/Alias.ts')),
new Import('./DoubleQuote', new File('resolver/ts/import-syntax/DoubleQuote.ts')),
new Import('./Simple', new File('resolver/ts/import-syntax/Simple.ts'))
new Import('./Simple', new File('resolver/ts/import-syntax/Simple.ts')),
], $imports->getImports());

self::assertEquals([], $imports->getResources());
Expand Down
1 change: 0 additions & 1 deletion test/Import/ImportFinderTest.php
Expand Up @@ -29,7 +29,6 @@ public function testAll()
$file = new File('foo.js');

$collector = new class implements ImportCollectorInterface {

public function supports(File $file): bool
{
return true;
Expand Down