Skip to content
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

add vscode repo #13

Merged
merged 3 commits into from
Jun 20, 2024
Merged

add vscode repo #13

merged 3 commits into from
Jun 20, 2024

Conversation

Boshen
Copy link
Member

@Boshen Boshen commented Jun 14, 2024

No description provided.

@Boshen
Copy link
Member Author

Boshen commented Jun 18, 2024

Still failing :-(

@rzvxa
Copy link

rzvxa commented Jun 18, 2024

I'll take care of it

@rzvxa
Copy link

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
Member Author

Boshen commented Jun 19, 2024

included the other cfg rules

@rzvxa
Copy link

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

rzvxa commented Jun 19, 2024

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

@Boshen Boshen merged commit c448d4e into main Jun 20, 2024
10 checks passed
@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