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.

Sign up for GitHub

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

Jump to bottom

Add support for dotnet publish #487

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add support for dotnet publish #487

wants to merge 4 commits into from

Conversation

zijchen
Copy link
Member

@zijchen zijchen commented Sep 6, 2024

Running dotnet publish will place the dacpac and all dependencies into the publish folder. Note this does not publish dacpac to databasae, please use sqlpackage for that.

Closes #447

@dzsquared
Copy link
Contributor

clarifying "and all dependencies" - if I have a sqlproj with a dacpac file reference and a dacpac nupkg reference - then run dotnet publish -
is the result that the project.dacpac is placed in the output/publish folder plus the other 2 dacpacs?

@zijchen
Copy link
Member Author

zijchen commented Sep 9, 2024

clarifying "and all dependencies" - if I have a sqlproj with a dacpac file reference and a dacpac nupkg reference - then run dotnet publish - is the result that the project.dacpac is placed in the output/publish folder plus the other 2 dacpacs?

That is correct.

@llali
Copy link
Member

llali commented Sep 13, 2024

does publish folder include database references? master dacpacs?

@zijchen
Copy link
Member Author

zijchen commented Sep 13, 2024

does publish folder include database references? master dacpacs?

Yes, all references, whether they come from a project, Nuget, or direct dacpac reference get copied to publish folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llali llali llali approved these changes

@abhikum abhikum Awaiting requested review from abhikum

@dzsquared dzsquared Awaiting requested review from dzsquared

@chlafreniere chlafreniere Awaiting requested review from chlafreniere

@ssreerama ssreerama Awaiting requested review from ssreerama

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support dotnet publish
3 participants
@zijchen @dzsquared @llali