Skip to content

Improve integration with source link#92

Merged
kzu merged 2 commits intomainfrom
early-scc
May 10, 2021
Merged

Improve integration with source link#92
kzu merged 2 commits intomainfrom
early-scc

Conversation

@kzu
Copy link
Copy Markdown
Member

@kzu kzu commented May 10, 2021

No description provided.

@kzu kzu enabled auto-merge (rebase) May 10, 2021 16:19
kzu added 2 commits May 10, 2021 13:29
Ensure properties are populated before source generators and assembly info is determined.

Also shorten the commit to the 9-char version as RepositorySha, and update SourceRevisionId unless opted-out.

Fixes #56
If SourceLink is used, the RepositoryUrl will be initialized from the PrivateRepositoryUrl (as long as PublishRepositoryUrl=true). This URL should also be the default for PackageProjectUrl since it's quite common for OSS projects to not have a separate site.

Fixes #91
@kzu kzu merged commit d942404 into main May 10, 2021
@kzu kzu deleted the early-scc branch May 10, 2021 16:37
@devlooped devlooped locked and limited conversation to collaborators Nov 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant