search

ArtitalkJS / Artitalk

通过leancloud实现的可实时发布说说的js
https://artitalk.js.org
MIT License
329 stars 52 forks source link

perf: improve gulp performance #42

Closed ChrAlpha closed 4 years ago

ChrAlpha commented 4 years ago

this PR mainly about:

ChenYFan commented 4 years ago

啊哈？有优化当然好，我就是那种分秒必争的人。 合并看drew心情扒，我先看看你改了什么

ChenYFan commented 4 years ago

另外麻烦合并到dev分支里，合并到master分支总不太好。。。

so1ve commented 4 years ago

但是我觉得有必要用rollup

ChrAlpha commented 4 years ago

另外麻烦合并到dev分支里，合并到master分支总不太好。。。

Dev 分支比 master 分支慢 20+ 个 commit，这很难让人理解。

ChenYFan commented 4 years ago

看到了，dev就是拿来吃灰的。。。

jalenzz commented 4 years ago

nice 学习了

ChrAlpha commented 4 years ago

还有些细节没有在此 PR 中体现，例如兼容参数我大概会这么写：

    autoprefixer: {
        overrideBrowserslist: [
            'last 2 versions',
            '> 1%',
            'Chrome >= 40',
            'Firefox >= 40',
            'ie >= 10',
            'Safari >= 8'
        ]
    },

但是你们似乎想兼容至 ie8，那还是不打扰了。

ChenYFan commented 4 years ago

我们在QQ群教训了drew一顿，但是drew忘记给我们writeaccess了。。。等drew回来再合并扒 另外欢迎前往QQ群灌水 讨论：1104585229

jalenzz commented 4 years ago

@ChrAlpha 不是想兼容 ie8，那个是直接从插件 README 里复制过来的，就没有删掉看

so1ve commented 4 years ago

主要是现在这个项目还处在挺低级的阶段， 没有linter， 没有打包器， 没有css预处理器， 全部都是直接操作dom， 还没用es6的语法 有必要大改了……

so1ve commented 4 years ago

什么autoprefixer、browserlist都没用

ChrAlpha commented 4 years ago

主要是现在这个项目还处在挺低级的阶段， 没有linter， 没有打包器， 没有css预处理器， 全部都是直接操作dom， 还没用es6的语法 有必要大改了……

我还有准备另一个 PR，修复命名污染的。全部用全局变量说实话有点 dirty。 昨晚就写好了，现在还在测试，要不我先发 draft PR 让大家一起测？

ChenYFan commented 4 years ago

artitalk还是一个较为前期的个人项目，drew是这个项目的创始人，我们尤其希望大佬来提意见。 gulp使用压缩虽然是个小事，但这种求细节的想法也值得作为项目开发的理念。这个pr帮助我们暴露出种种细节的问题，感谢pr！

so1ve commented 4 years ago

可以fork这个仓库到一个组织的新仓库里做测试

ChrAlpha commented 4 years ago

可以fork这个仓库到一个组织的新仓库里做测试

直接自定义路由然后拉取这个 PR 到本地测试就行了

so1ve commented 4 years ago

可以fork这个仓库到一个组织的新仓库里做测试

直接自定义路由然后拉取这个 PR 到本地测试就行了

嗯

Drew233 commented 4 years ago

感谢pr，因为我也是从什么都不会的阶段开始写的at，不合适，不对的地方肯定有很多，感谢大佬指正。希望以后多多指点🙏

ChrAlpha commented 4 years ago

感谢pr，因为我也是从什么都不会的阶段开始写的at，不合适，不对的地方肯定有很多，感谢大佬指正。希望以后多多指点🙏

我也是自己一点点慢慢摸索来的，实在称不上大佬。