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
Feature/scratch from scratch #2
Conversation
# Conflicts: # paket.dependencies # paket.lock
…c/bob-scratch into feature/scratch-from-scratch
Feature/6 scratch fixes
Fix collectionuser creation
@@ -0,0 +1,95 @@ | |||
<# |
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.
Can we rename this to "ScratchHostHeaderTask, please?
the SQL server. | ||
|
||
.EXAMPLE | ||
Enable-ScSite |
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.
Example does not really make sense i guess :)
Unic.Bob.Scratch.nuspec
Outdated
<requireLicenseAcceptance>false</requireLicenseAcceptance> | ||
<description>Scratch install Sitecore with SIF.</description> | ||
</metadata> | ||
<files> | ||
<file src="Installation.config" /> |
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.
if we manage to move/remove installation.config, this entry is not required anymore.
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.
Just a general remark belonging to almost everything. The comments and description of the individuell cmdlets is currently just something. It does not respect to the cmdlets and theyr doing in most cases. So this should be rechecked globally and adapted.
src/Scratch/Install-Sitecore12.ps1
Outdated
@@ -0,0 +1,78 @@ | |||
<# |
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.
Needs a better file name and an updated doc header. Maybe Install-SitecoreDevInstance?
@@ -0,0 +1,48 @@ | |||
<# |
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.
Eventually Install-SitecoreDevInstanceDatabases would make sense. Not sure if you guys plan to use this wrapper for any other situations than local dev? (and if it is only used on dev, do we need it?)
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.
no more relevant feedbacks. I added the corrections myself.
No description provided.