metals icon indicating copy to clipboard operation
metals copied to clipboard

docs: Add a design doc for Language Support for notebook cells

Open tanishiking opened this issue 1 year ago • 2 comments

It would be easier to read with rendered one :) https://github.com/scalameta/metals/blob/ccd1bed3a6eabe0d67d401be18ba348d1c494f3a/docs/design/language-support-notebook.md

Added a design document for https://github.com/scalameta/metals-feature-requests/issues/236 This document is for

  • design review the architecture before going further
  • make sure we are on the same page about the goal, design, and requirements for this feature

This PR is just for the design review. I opened a PR because it's easier to review (maybe we could write a design doc on Google Docs or something). Therefore, I don't need to merge this PR after people approved the architecture design, but this document could be useful as a log in the future.

tanishiking avatar Aug 30 '22 10:08 tanishiking

Can you take a look, and give me some thoughts? @dos65 @tgodzik @alexarchambault (requesting review to someone who commented on the original feature requests).

Maybe you can skip until Problem section because I guess you guys already know about the context.

tanishiking avatar Aug 31 '22 06:08 tanishiking

@alexarchambault Hi, could you skim through this doc when you have time before I'm going further (to make sure we're on the same page about the implementation plan) I'm not in a rush (as I might not have time to work on this for several weeks), take you time :)

tanishiking avatar Sep 19 '22 10:09 tanishiking

I'm only not sure if there is need in adding it to metals docs. Maybe it might be left in a issue comment to this feature request?

As @dos65 mentioned, we don't need to merge this PR (I just submit PR because it seems easier to review), I'll turn this into an issue and closing :)

tanishiking avatar Sep 23 '22 14:09 tanishiking

https://github.com/scalameta/metals/issues/4434

tanishiking avatar Sep 23 '22 14:09 tanishiking