-
-
Notifications
You must be signed in to change notification settings - Fork 452
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
release: 1.6.0 #1969
release: 1.6.0 #1969
Conversation
✅ Deploy Preview for biomejs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I am working on rule promotion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice post! Just did a bit of proofreading.
@@ -63,7 +63,7 @@ impl VueFileHandler { | |||
|
|||
/// Returns the start byte offset of the Vue `<script>` tag | |||
pub fn start(input: &str) -> Option<u32> { | |||
Self::matches_script(input).map(|m| m.start() as u32 - 1) | |||
Self::matches_script(input).map(|m| m.start() as u32) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fixes an issue in the LSP when showing diagnostics. Spans were incorrect
Co-authored-by: Arend van Beelen jr. <[email protected]>
CodSpeed Performance ReportMerging #1969 will not alter performanceFalling back to comparing Summary
|
hello guys, I'm assuming this will include all the nightly release comments? If so then please note that there seem to be an error in the very first nightly release comment (from 3 hours ago) I'm assuming |
thank you @ghiscoding , this is fixed in |
Parser conformance results onjs/262
jsx/babel
symbols/microsoft
ts/babel
ts/microsoft
|
Co-authored-by: Victorien Elvinger <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! some small nits:
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Co-authored-by: Superchupu <[email protected]>
Summary
Test Plan
Proof read https://deploy-preview-1969--biomejs.netlify.app/