Skip to content
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

Check list for creating new Arcade package #374

Merged
merged 5 commits into from
Aug 2, 2018
Merged

Check list for creating new Arcade package #374

merged 5 commits into from
Aug 2, 2018

Conversation

JohnTortugo
Copy link
Contributor

Closes: #370

@maririos
Copy link
Member

It might be worth to add that a way to pack the package is by using dotnet pack like what @alexperovich is promoting in #380

@@ -0,0 +1,39 @@
# How to Create an Arcade Package

One of the goals of Arcade is to be a vehicle to provide code sharing. One of the ways that this goal is implemented by through NuGet packages. This document assumes that you are adding a package called `ProjectOne` to Arcade and provides a list of things you should check when while doing that.

This comment was marked as spam.

@@ -0,0 +1,39 @@
# How to Create an Arcade Package

One of the goals of Arcade is to be a vehicle to provide code sharing. One of the ways that this goal is implemented by through NuGet packages. This document assumes that you are adding a package called `ProjectOne` to Arcade and provides a list of things you should check when while doing that.

This comment was marked as spam.


`src/Microsoft.DotNet.ProjectOne/tests`

- It would be great if you could include a `README.md` file on the root of the package.

This comment was marked as spam.

This comment was marked as spam.


- Include the package on the `Arcade.sln` file. This way it will be compiled automatically when the solution is built.

- Make sure that library dependencies of the new package use the Arcade defined version of the library. The versions are defined in the `eng\Version.props` file. For instance, if the project has a dependence on Newtonsoft.Json you must add it as follows:

This comment was marked as spam.

@JohnTortugo
Copy link
Contributor Author

@maririos pushed new changes addressing your comments. Thanks.

I believe the packages are created automatically during the building process. Do you want me to include a comment about how to manually create the package on the command line?

## Further References

- https://github.com/dotnet/arcade/blob/master/Documentation/Overview.md#toolset-nuget-package-requirements

This comment was marked as spam.

@JohnTortugo JohnTortugo merged commit 2861456 into dotnet:master Aug 2, 2018
@JohnTortugo JohnTortugo deleted the PackageHowTo branch August 2, 2018 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants