Skip to content

Add function to create a framework dependent package dotnet-sdk containers - #8644

Merged
Aditya Patwardhan (adityapatwardhan) merged 4 commits into
PowerShell:masterfrom
adityapatwardhan:buildtest
Jan 16, 2019
Merged

Add function to create a framework dependent package dotnet-sdk containers#8644
Aditya Patwardhan (adityapatwardhan) merged 4 commits into
PowerShell:masterfrom
adityapatwardhan:buildtest

Conversation

@adityapatwardhan

Copy link
Copy Markdown
Member

PR Summary

Create a framework dependent package with reduced size specially for docker containers based on dotnet-sdk containers.

PR Context

The size of the framework dependent package is pretty large. It has assemblies of runtimes which are not required for the dotnet-sdk container images. This PR creates a new package with a reduced size.

PR Checklist

Comment thread tools/packaging/packaging.psm1 Outdated
Comment thread tools/packaging/packaging.psm1
Comment thread tools/packaging/packaging.psm1 Outdated
Comment thread tools/packaging/packaging.psm1

@TravisEz13 Travis Plunk (TravisEz13) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would like you to fix the Write-Log comment before merging.

@adityapatwardhan

Copy link
Copy Markdown
Member Author

Travis Plunk (@TravisEz13) can you re-review, your comments have been addressed.

@adityapatwardhan

Copy link
Copy Markdown
Member Author

Andrew (@anmenaga) - Please have a look I have made changes to package verification tests.

@adityapatwardhan

Copy link
Copy Markdown
Member Author

Codacy error is due to Write-Host which is needed.
CodeFactor is complaining about docker files. Similar issues exists in all our docker files.

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit be4b82c into PowerShell:master Jan 16, 2019
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants