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

Remove support for "dnu bundle" #1545

Merged
merged 1 commit into from
Mar 30, 2015
Merged

Conversation

ChengTian
Copy link
Contributor

@ghost ghost added the cla-not-required label Mar 30, 2015
@ChengTian
Copy link
Contributor Author

Warning for legacy name bundleExclude is added and tested:
capture

}

if (missingOutputFolder.Any())
{
throw new InvalidOperationException("Bundled output does not contain directory:\n" + string.Join("\n", missingOutputFolder));
throw new InvalidOperationException("Publishd output does not contain directory:\n" + string.Join("\n", missingOutputFolder));
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo in the exception

@muratg
Copy link
Contributor

muratg commented Mar 30, 2015

Fix the typos, and :shipit:

- Change 'bundle' to 'publish' in class/method names and messages
- Add warning for deprecated name "bundleExclude"
@ChengTian ChengTian force-pushed the wei/remove-support-for-dnu-bundle branch from b54131b to f4a5a27 Compare March 30, 2015 21:45
@ChengTian ChengTian merged commit f4a5a27 into release Mar 30, 2015
@ChengTian ChengTian deleted the wei/remove-support-for-dnu-bundle branch March 30, 2015 22:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants