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

Add spire to community build #13788

Merged
merged 5 commits into from
Nov 18, 2021
Merged

Add spire to community build #13788

merged 5 commits into from
Nov 18, 2021

Conversation

armanbilge
Copy link
Contributor

@armanbilge
Copy link
Contributor Author

Blocked by #13860.

@armanbilge
Copy link
Contributor Author

Green! Can someone kindly help get this merged, before there is another regression? 😆

.gitmodules Outdated
@@ -216,3 +216,6 @@
[submodule "community-build/community-projects/specs2"]
path = community-build/community-projects/specs2
url = https://github.com/dotty-staging/specs2.git
[submodule "community-build/community-projects/spire"]
path = community-build/community-projects/spire
url = https://github.com/typelevel/spire.git
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
url = https://github.com/typelevel/spire.git
url = https://github.com/dotty-staging/spire.git

As a rule we use links to forks instead of original repos to ensure reproducibility of builds from the compiler's perspective (if the main branch of the community project repo changes, possibly making the community build fail, but the compiler's master doesn't change then master's build should still succeed if it succeeded earlier).
I created the fork so this should work with the suggested change

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

2 participants