-
Notifications
You must be signed in to change notification settings - Fork 4k
DSC extension refactored code for ARM & ASM #647
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
Merged
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
f910122
Support for side by side module versioning
eshaparmar e2e95e4
version conditions got swiped by mistake
eshaparmar 783ab7a
CR comments
eshaparmar 4e319ad
Added format file for dsc get cmdlet in ARM
eshaparmar af44e43
Merge branch 'dev' of https://github.com/dscsolutions/azure-powershel…
eshaparmar d36f516
Merge branch 'armcmdlets-2.0' into bugfixes
eshaparmar 3e5d8f3
Fix storage suffix in ASM cmdlets
narrieta 15cfa04
Merge branch 'armcmdlets' of https://github.com/dscsolutions/azure-po…
narrieta 1c018ae
Added storage suffix to ARM cmdlets
narrieta 80f8a1e
Move common code to commons
eshaparmar 817d3e0
Merge branch 'armcmdlets' into dev
eshaparmar 23a7810
more changes
eshaparmar 7c8f76c
Merge branch 'dev' of https://github.com/dscsolutions/azure-powershel…
narrieta 3b5d5f6
Merge pull request #4 from dscsolutions/storage-suffix
narrieta c9ebb91
more changes
eshaparmar 7ea546d
renamed extensions folder and added vs proj settings
eshaparmar d6c432b
Correcting paths and vs proj file
eshaparmar 548a2ba
Fixed the build issues
eshaparmar 21930a7
End to end publish ASM workflow
eshaparmar 384850b
Final refactoring for DSC code for ARM and ASM
eshaparmar c143ebe
Fixed resharper warnings from serialier class
eshaparmar 7c9083b
Merge storage suffix changes from dscsolutions 'dev' to 'refactoring'
eshaparmar e0e431a
Removed public and private settings from get output format file
eshaparmar bf25298
Making cmdlet base classes internal
eshaparmar 973db34
Changing methods to internal
eshaparmar 8c4427c
Merge branch 'dev' of https://github.com/Azure/azure-powershell into …
eshaparmar 9fae049
Merge branch 'dev' of https://github.com/dscsolutions/azure-powershel…
eshaparmar e27e3b8
Fixing merge conflicts
eshaparmar 18b9182
Adding extension name and publisher to get/remove ASM cmdlets that go…
eshaparmar 593276a
Regenerated wxi file
eshaparmar 2484821
Merge branch 'dev' of https://github.com/Azure/azure-powershell into dev
eshaparmar 166d48e
Merge branch 'dev' into dsc_release
eshaparmar 5d4f8e9
Regenerated wix file post merge from dev
eshaparmar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
24 changes: 24 additions & 0 deletions
24
src/Common/Commands.Common/Extensions/DSC/DscExtensionCmdletConstants.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
using System; | ||
using System.Collections.Generic; | ||
|
||
namespace Microsoft.WindowsAzure.Commands.Common.Extensions.DSC | ||
{ | ||
public static class DscExtensionCmdletConstants | ||
{ | ||
//common extension constants | ||
internal static readonly string ExtensionPublishedNamespace = "Microsoft.Powershell"; | ||
internal static readonly string ExtensionPublishedName = "DSC"; | ||
internal const string DefaultContainerName = "windows-powershell-dsc"; | ||
|
||
internal const int MinMajorPowerShellVersion = 4; | ||
internal const string ZipFileExtension = ".zip"; | ||
internal const string Ps1FileExtension = ".ps1"; | ||
internal const string Psm1FileExtension = ".psm1"; | ||
|
||
|
||
internal static readonly HashSet<String> UploadArchiveAllowedFileExtensions = | ||
new HashSet<String>(StringComparer.OrdinalIgnoreCase) { Ps1FileExtension, Psm1FileExtension, ZipFileExtension }; | ||
internal static readonly HashSet<String> CreateArchiveAllowedFileExtensions = | ||
new HashSet<String>(StringComparer.OrdinalIgnoreCase) { Ps1FileExtension, Psm1FileExtension }; | ||
} | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you've added a new DLL to the output. Please regenerate WIX file if that is the case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure why we need to regenerate the wix file. We haven't added any new project/assembly. Just re-using an existing one(Microsoft.WindowsAzure.Management) and added it as a reference to the Command.Common project.