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

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

Open
wants to merge 20 commits into
base: 8.x-3.x-dev
Choose a base branch
from

Conversation

tjwelde
Copy link
Contributor

@tjwelde tjwelde commented Feb 20, 2018

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

const CircularDependencyPlugin = require('circular-dependency-plugin')
Copy link
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
Contributor

Choose a reason for hiding this comment

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

Thanks!

Copy link
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
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