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

Molving to preprocessify instead of remove lines since it's more vers… #37

Merged
merged 2 commits into from
Oct 15, 2015

Conversation

arcturus
Copy link
Contributor

@arcturus arcturus commented Oct 9, 2015

…atile

Also checking if we have performance.mark in order to not to fail in chrome

…atile

Also checking if we have performance.mark in order to not to fail in chrome
@arcturus
Copy link
Contributor Author

arcturus commented Oct 9, 2015

@lodr r?

@@ -26,19 +26,15 @@ gulp.task('bundle-dist', function() {
var bundler = browserify({
entries: ['./index.js'],
debug: false,
standAlone: 'ServiceWorkerWare'
standAlone: 'ServiceWorkerWare',
Copy link
Contributor

Choose a reason for hiding this comment

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

Beware of that colon at the end of the line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

right!

@delapuente
Copy link
Contributor

I like this preprocessor. It's a little bit verbose to enclose all with the directives and it would be great to have a special syntax for the directives to affect next line / block only.

arcturus added a commit that referenced this pull request Oct 15, 2015
Molving to preprocessify instead of remove lines since it's more vers…
@arcturus arcturus merged commit f08c819 into master Oct 15, 2015
@arcturus arcturus deleted the preprocessify branch October 15, 2015 14:03
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.

2 participants