-
Notifications
You must be signed in to change notification settings - Fork 1.1k
make it clear in documentation that ./scala-scala/ is required #2132
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
Comments
Dotty doc is getting repl checking similar to tut - as soon as I've cleared away other more pressing tasks. We could make the tests depend on running a script to make sure scala-scala is cloned and updated, I did something similar when dottydoc had a Scala.js client. |
What's the argument against dumping everything into /tests/scala-library? |
I've argued in the past that this is what we eventually should do - I don't think the changes we're applying really have anything to do with improving, or making the old collections compatible with Dotty. Especially now that we are including the new collections in 2.13. They're just a good set of integration tests IMO and should be moved to |
It can also be automated, you know ;) |
@sjrd: omg, no more sbt magic please 😂 |
That's a rare case when I'd prefer sbt magic to the alternative. |
Did the reorg break one of your tool in particular? The only change for me is that I have to remember to pass |
I don't think we'll be upstreaming changes anyway as the new library is on its way in, and if so - it feels like the other arguments fall away. We could still filter in the changes from the forked library, that would solve the problem of relevant history. Clarification: I'm not proposing to include the |
It broke both IntelliJ, github and SourceTree.
Note that standard library has more to it than standard collections. But that's true that collections represent >80% of it. |
That's true - but what else are we interested in compiling? |
I think it'd be good to try to compile as much of scala-library as possible. |
Indeed, I didn't notice github didn't --follow, they seem to be aware of the issue at least: isaacs/github#900 |
Mostly off-topic: https://chrome.google.com/webstore/detail/github-follow/agalokjhnhheienloigiaoohgmjdpned there's probably an alternative for FF too |
According to the Internet, Sourcetree has a checkbox to enable follow, but they don't have a Linux version so I can't test it. |
Well, the argument can still be made that this is non-obvious to contributors - we should document it. |
Have either of these ever append since the fork?
In the current situation if I want to understand history I have to manually correlate timestamps between two repos. Also, I don't really see the value of commit history anterior to the fork point for tests file. If I make an analogy with any other library it sounds really silly: suppose I want to add parts of library
? |
No, but it's a bad thing on our side. There has been substantial improvements in Scala standard library over last 2 years.
Making an analogy here is wrong, this is the standard library which is our dependency. It isn't just a random test, it's part of out bootstrap. Unlike any other library\test it is also being actively developed. |
Now that |
"Getting started" guide doesn't say anything about it.
Idea: integrate "getting started" with something similar to "tut"?
The text was updated successfully, but these errors were encountered: