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

add roadmap override check #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hefangshi
Copy link
Member

关联fex-team/fis#299 效果如下

fis.config.merge({
    roadmap: {
        path: [
            {
                reg: '**.css',
                release: '/static/$&'
            },
            {
                reg: '**.js',
                release: '/static/$&'
            }
        ]
    }
});

fis.config.merge({
    roadmap: {
        path: [
            {
                reg: '**.css',
                release: '/static/$&'
            }
        ]
    }
});

fis.config.set('roadmap.path', [
    {
        reg: '**.js',
        release: '/static/$&'
    }
]);

fis.config.set('roadmap', {
    path: [
    {
        reg: '**.js2',
        release: '/static/$&'
    }
]
});
D:\Work\Code\test\override>fis release

 [WARNI] roadmap.path [**.css, **.js] was overrided by [**.css] at D:\Work\Code\
test\override\fis-conf.js:16:12, please use fis.config.get('roadmap.path').unshi
ft(ruler) to add rulers.


 [WARNI] roadmap.path [**.css, **.js] was overrided by [**.js] at D:\Work\Code\t
est\override\fis-conf.js:27:12, please use fis.config.get('roadmap.path').unshif
t(ruler) to add rulers.


 [WARNI] roadmap.path [**.js] was overrided by [**.js2] at D:\Work\Code\test\ove
rride\fis-conf.js:34:12, please use fis.config.get('roadmap.path').unshift(ruler
) to add rulers.

@oxUnd
Copy link
Contributor

oxUnd commented Jan 22, 2015

这个事情由各自的解决方案负责,内核不做处理。

@2betop
Copy link
Contributor

2betop commented Jan 23, 2015

jello里面确实已用另外一种方式处理了!就是解决方案里面的roadmap在release前才合并进来

@oxUnd
Copy link
Contributor

oxUnd commented Jan 23, 2015

最好的办法是,用另外一种机制保证,提示终归是下下选。这块再考虑考虑吧。

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.

3 participants