Skip to content

Conversation

@bishabosha
Copy link
Member

this is the initial version of the Scala 3 contributing guide, optimised for dotty sprees.

Copy link
Member

@sjrdsjrd left a comment

Choose a reason for hiding this comment

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

Reviewed "Procedures" so far.

Copy link
Member

@sjrdsjrd left a comment

Choose a reason for hiding this comment

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

And that's the review for the Architecture.

Copy link
Contributor

@vincenzobazvincenzobaz left a comment

Choose a reason for hiding this comment

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

The other reviewers probably have a lot more knowledge than me on this topic.
I wish I had this before my first spree!

I left some minor comments, but the guide looks very good and it contains a lot cool tips on testing. For the Spree I suggest to create a resume or shorter version, because there is a lot of material!

│ ├── ast // Abstract syntax trees
│ ├── classpath
│ ├── config // Compiler configuration, settings, platform specific definitions.
│ ├── core // Core data structures and operations, with specific subpackages for:
Copy link
Contributor

Choose a reason for hiding this comment

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

Earlier you mentioned a few phases (typer, parser) and @sjrd mentioned erasure...
Here I only see parser and typer. Where are the other phases?

part when you click it, or instrumenting variables for printing.

An example of how it is done for Sublime Text 3 is [here](https://github.com/anatoliykmetyuk/scala-debug-sublime).

Copy link
Contributor

Choose a reason for hiding this comment

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

Do other people in the team want to share their editor setup? I imagine most of them use VSCode so it would be nice to have helpers for it as well

@anatoliykmetyukanatoliykmetyuk merged commit b9e014e into scala:mainOct 1, 2021
@anatoliykmetyukanatoliykmetyuk deleted the add-scala-3-contributing-guide branch October 1, 2021 14:30
@anatoliykmetyuk
Copy link
Contributor

Thanks everyone for the review and suggestions! I've applied the most straightforward ones while leaving more in-depth one for later. This is because we have only a couple days left until the spree, so we need to have something soon. Let's return to this PR in the future iterations of the development of this guide.

Sign up for freeto 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.

6 participants

@bishabosha@anatoliykmetyuk@SethTisue@dwijnand@sjrd@vincenzobaz