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

Updates project settings so it can generate nuget packages for the Testing library. #2078

Merged
merged 2 commits into from
Jun 14, 2019

Conversation

gabog
Copy link
Contributor

@gabog gabog commented Jun 14, 2019

Added true to the csproj file so it can generate nuget packages from the testing library.
Removed unused ruleset and xml file.
Updated csproj settings to match BotBuilder csproj

See (this)[https://stackoverflow.com/questions/48320344/cannot-create-nuget-package-that-has-xunit-as-dependency] for the solution for nuget packages not being generated for projects that reference XUnit or MSTest

…rate nuget packages from the testing library.

Removed unused ruleset and xml file.
Updated csproj settings to match BotBuilder csproj
@gabog gabog requested a review from cleemullins June 14, 2019 19:30
@coveralls
Copy link
Collaborator

coveralls commented Jun 14, 2019

Pull Request Test Coverage Report for Build 65079

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 71.687%

Totals Coverage Status
Change from base Build 65051: 0.0%
Covered Lines: 4512
Relevant Lines: 6294

💛 - Coveralls

@fuselabs
Copy link
Collaborator

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Configuration.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Connector.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Schema.dll

Copy link
Contributor

@cleemullins cleemullins left a comment

Choose a reason for hiding this comment

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

Per this GitHub thread:
xunit/xunit#1520

You should not be importing XUnit. Instead they are recommending:

As a reminder: If you're extending xUnit.net and want to publish your extension as a NuGet package, you should import xunit.extensibility.core and/or xunit.extensibility.execution, not xunit or xunit.core. If you do this wrong, you might have problems generating your NuGet package via dotnet pack.

…t and removed the property to force packaging (is not needed anymore)
@gabog gabog requested a review from cleemullins June 14, 2019 20:41
@fuselabs
Copy link
Collaborator

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Configuration.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Connector.dll
✔️ No Binary Compatibility issues for Microsoft.Bot.Schema.dll

@cleemullins cleemullins merged commit 2053b64 into master Jun 14, 2019
@cleemullins cleemullins deleted the gabog/MakeTestingLibraryPackageable branch June 14, 2019 21:11
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.

None yet

4 participants