chore: tweaks & fixes (#9109)

* fix: correct bundle analyzer settings

* fix: svelte config fix

* fix: CopyWebpackPlugin fixed ignore

* chore: tabs fix
This commit is contained in:
Martin Guillon
2020-12-23 20:04:22 +01:00
committed by GitHub
parent de5f67f7c8
commit 68f3e728a6
3 changed files with 21 additions and 10 deletions

View File

@@ -18,6 +18,8 @@ import {
getEntryDirPath,
getEntryPath,
} from '../helpers/platform';
import { getProjectRootPath } from '../helpers/project';
import { resolve } from 'path';
export default function (config: Config, env: IWebpackEnv): Config {
const entryPath = getEntryPath();
@@ -266,7 +268,14 @@ export default function (config: Config, env: IWebpackEnv): Config {
});
config.when(env.report, (config) => {
config.plugin('BundleAnalyzerPlugin').use(BundleAnalyzerPlugin);
const projectRoot = getProjectRootPath();
config.plugin('BundleAnalyzerPlugin').use(BundleAnalyzerPlugin, [{
analyzerMode: 'static',
generateStatsFile: true,
openAnalyzer: false,
reportFilename: resolve(projectRoot, 'report', 'report.html'),
statsFilename: resolve(projectRoot, 'report', 'stats.json'),
}]);
});
return config;

View File

@@ -17,7 +17,6 @@ export default function (config: Config, env: IWebpackEnv = _env): Config {
// resolve .svelte files
// the order is reversed because we are using prepend!
config.resolve.extensions.prepend('.svelte').prepend(`.${platform}.svelte`);
// add a rule for .svelte files
config.module
.rule('svelte')
@@ -30,7 +29,7 @@ export default function (config: Config, env: IWebpackEnv = _env): Config {
return {
...options,
dev: !production,
preprocess: [getSvelteConfigPreprocessor(), svelteNativePreprocessor()],
preprocess: getSvelteConfigPreprocessor(),
hotReload: !production,
hotOptions: {
injectCss: false,

View File

@@ -1,6 +1,8 @@
import CopyWebpackPlugin from 'copy-webpack-plugin';
import { relative, resolve } from 'path';
import { env } from '..';
import { getEntryDirPath } from './platform';
import { getProjectRootPath } from './project';
/**
* @internal
@@ -36,17 +38,18 @@ export function removeCopyRule(glob: string) {
* @internal
*/
export function applyCopyRules(config) {
const context = getEntryDirPath();
const projectRoot = getProjectRootPath();
const appResourcesFullPath = resolve(projectRoot, env.appResourcesPath);
const globOptions = { dot: false, ignore: [`**/${relative(context, appResourcesFullPath)}/**`] };
config.plugin('CopyWebpackPlugin').use(CopyWebpackPlugin, [
{
patterns: Array.from(copyRules).map((glob) => ({
from: glob,
context: getEntryDirPath(),
context,
noErrorOnMissing: true,
globOptions: {
dot: false,
// todo: ignore AppResources if inside app folder!
// ignore: [``]
},
globOptions,
})),
},
]);