Skip to content
This repository was archived by the owner on Nov 14, 2023. It is now read-only.

[WIP] INREL-3418 Refactor & Bundle JS#92

Open
tjwelde wants to merge 20 commits into
8.x-3.x-devfrom
feature/INREL-3418-refactor-bundle-js
Open

[WIP] INREL-3418 Refactor & Bundle JS#92
tjwelde wants to merge 20 commits into
8.x-3.x-devfrom
feature/INREL-3418-refactor-bundle-js

Conversation

@tjwelde
Copy link
Copy Markdown
Contributor

@tjwelde tjwelde commented Feb 20, 2018

Comment thread webpack.dev.js Outdated
const merge = require('webpack-merge');
const common = require('./webpack.common.js');

const CircularDependencyPlugin = require('circular-dependency-plugin')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's smart!


/**
* Load blazy shizzl
* get model and track impression
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The diff is just wrong. Didn't update comments ;)

@saviomuc
Copy link
Copy Markdown
Contributor

As discussed let's move the config (package.json, webpack conf, etc.) to custom.

If we feel the need to move it back to contrib later (shared config on all other projects) we can think about a smarter approach.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants