Skip to content

Fix for #3494 - #3534

Merged
forki merged 8 commits into
fsprojects:masterfrom
jam40jeff:master
Apr 2, 2019
Merged

forki merged 8 commits into
fsprojects:masterfrom
jam40jeff:master

Conversation

@jam40jeff

@jam40jeff jam40jeff commented Mar 27, 2019 •

Copy link
Copy Markdown
Contributor

This fixes #3494 by adding support for multiple target frameworks to the pack command.

@jam40jeff

Copy link
Copy Markdown
Contributor Author

Sorry, but I don't use git very often. I tried to clean up the pull request with a rebase, but it looks like it just made more commits. The diff still looks correct, however. Feel free to instruct me to cleanup if necessary.

@jam40jeff

Copy link
Copy Markdown
Contributor Author

This should be ready for review. It looks like only the flaky tests are failing.

@forki

forki commented Apr 2, 2019

Copy link
Copy Markdown
Member

can you please merge with master?

@jam40jeff

Copy link
Copy Markdown
Contributor Author

@forki Done.

@forki

forki commented Apr 2, 2019

Copy link
Copy Markdown
Member

cool. do we need to add something to the docs?

@jam40jeff

Copy link
Copy Markdown
Contributor Author

I don't believe so. Nothing public-facing changed and IMO it works as one would expect now when multi-targeting.

@forki
forki merged commit 77815c7 into fsprojects:master Apr 2, 2019
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.

paket pack with multi-targeted project only creates first framework specified

2 participants