Skip to content
This repository has been archived by the owner on Dec 18, 2017. It is now read-only.

Remove "dnu bundle" and only use "dnu publish" #1474

Closed
ChengTian opened this issue Mar 24, 2015 · 4 comments
Closed

Remove "dnu bundle" and only use "dnu publish" #1474

ChengTian opened this issue Mar 24, 2015 · 4 comments
Assignees
Milestone

Comments

@ChengTian
Copy link
Contributor

#1428

@davidfowl , seems we also need to change bundleExclude to publishExclude. Do we want to keep support for bundleExclude? My suggestion is "No".

@muratg , we need to coordinate with Tooling team before making this change. So this will get in after release branching. I'll ask for approval before checking in.

@ChengTian ChengTian added the bug label Mar 24, 2015
@ChengTian ChengTian self-assigned this Mar 24, 2015
@ChengTian ChengTian added this to the 1.0.0-beta4 milestone Mar 24, 2015
@muratg
Copy link
Contributor

muratg commented Mar 24, 2015

@ChengTian yup, publishExclude makes more sense than bundleExecute to me.

Also, /cc: @mayurid

@ChengTian
Copy link
Contributor Author

Tracking issue in Tooling repo: https://github.com/aspnet/Tooling-Internal/issues/216

@davidfowl
Copy link
Member

+1

@ChengTian
Copy link
Contributor Author

Got reminder from @davidfowl , we can show warning for deprecated bundleExclude in Tooling now. We should remember to use this feature when renaming

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants