Daiki Nishikawa comments

Results 29 comments of


                                            Daiki Nishikawa

Docker build issue finding libcudart

TF 2.4 depends on CUDA 11.0 and cudnn 8, so we need to update the base image from `nvidia/cuda:10.1-cudnn7-devel` to `nvidia/cuda:11.0-cudnn8-devel-ubuntu18.04`.

`for-direction`

Hi 👋🏻 I'd like to work on this one as my first contribution! Can I take this issue?

`for-direction`

@ematipico When I implement it, I'm thinking of naming this rule `use_valid_for_direction`. How do you think?

📎 TypeScript 4.7

I'll take the support for "extends constraints on infer type variables" 🚀

📎 Add `files` config option for `ignore`

I take this issue and created the draft PR! Currently, I'm thinking about How treat "ignores" options in files config and linter config(or formatter config) ``` { "files": { "ignore":...

🐛 Update parser to support `satisfies` keyword (Typescript 4.9+)

I'll try to take this issue!

feat: Support TS satisfies expression

> There's one failing test that needs updating before merging the PR After checking one failing test, I confirmed the tests will pass by commenting out [this line](https://github.com/rome/tools/blob/67a888158ace04201f5c1d97c5b8898bd89405bf/crates/rome_js_formatter/tests/check_reformat.rs#L71) in check_reformat.rs....

🐛 Rome VS Code Extension crashes by inputting Japanese

I could reproduce this issue. The log `rome rage` shows is here. The log said tower_lsp failed encoding response. ``` ├─385415ms INFO rome_lsp::server Starting Rome Language Server... ┐rome_cli::commands::daemon::Running Server{pid=13245} ├─50ms...

📎 TypeScript 4.9

I'll try to take Auto Accessors support 🤚