Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

External source maps #12

Merged
merged 1 commit into from
May 18, 2015
Merged

External source maps #12

merged 1 commit into from
May 18, 2015

Conversation

zeorin
Copy link
Contributor

@zeorin zeorin commented May 14, 2015

I've fixed the pleeease task to write the source map to an external file if the option inline: false is entered.

I've also noticed that to get this to work, I still had to specifiy the in and out css files.

Here's a copy of my pleeease initConfig section:

[...]
        pleeease: {
            dist: {
                options: {
                    autoprefixer: {'browsers': ['last 2 versions', 'ie 9', 'ie 10']},
                    in: 'build/styles/styles.css',
                    out: 'public/styles/styles.min.css',
                    sourcemaps: {
                        map: {
                            inline: false,
                            sourcesContent: true
                        }
                    }
                },
                files: {
                    'public/styles/styles.min.css': 'build/styles/styles.css'
                }
            }
        },
[...]

@danielhusar
Copy link
Owner

Thanks :)
Do you think you can add some test to cover this?

danielhusar added a commit that referenced this pull request May 18, 2015
@danielhusar danielhusar merged commit b45cf43 into danielhusar:master May 18, 2015
@danielhusar
Copy link
Owner

Anyway I merged it for now, available in 1.3.2

@zeorin zeorin deleted the sourcemap-external branch October 9, 2015 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants