Skip to content

add vscode repo#13

Merged
Boshen merged 3 commits intomainfrom
vscode
Jun 20, 2024
Merged

add vscode repo#13
Boshen merged 3 commits intomainfrom
vscode

Conversation

@Boshen
Copy link
Copy Markdown
Member

@Boshen Boshen commented Jun 14, 2024

No description provided.

@Boshen
Copy link
Copy Markdown
Member Author

Boshen commented Jun 18, 2024

Still failing :-(

@rzvxa
Copy link
Copy Markdown

rzvxa commented Jun 18, 2024

I'll take care of it

@rzvxa
Copy link
Copy Markdown

rzvxa commented Jun 19, 2024

@Boshen I've run the tests again and no sign of false positives, There are some rules-of-hooks errors but I think they are all expected. I guess from the day you created this PR till now I've fixed it, We can track the last CI run checkout commit if you want to know which change resulted in it being fixed.

@Boshen
Copy link
Copy Markdown
Member Author

Boshen commented Jun 19, 2024

included the other cfg rules

@rzvxa
Copy link
Copy Markdown

rzvxa commented Jun 19, 2024

I think all no-fallthrough diagnostics are accurate, I'll investigate them right away.
As for getter-return false positives I'm working on them in this PR oxc-project/oxc#3714, If I'm not mistaken this particular one is already fixed, I'll check it out and if it is I'll extract it to its own PR so it can be merged sooner.

@rzvxa
Copy link
Copy Markdown

rzvxa commented Jun 19, 2024

you should merge this first oxc-project/oxc#3714

@Boshen Boshen merged commit c448d4e into main Jun 20, 2024
@Boshen Boshen deleted the vscode branch June 20, 2024 07:09
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.

2 participants