Skip to content

Improve preprocess#412

Merged
evoactivity merged 2 commits intoember-tooling:mainfrom
fisker:preprocess
Nov 30, 2025
Merged

Improve preprocess#412
evoactivity merged 2 commits intoember-tooling:mainfrom
fisker:preprocess

Conversation

@fisker
Copy link
Copy Markdown
Contributor

@fisker fisker commented Nov 30, 2025 •
edited
Loading

  • Replace all characters except \n with space, so the line/column number can remain correct.
  • Use replaceContents to deal with utf16 range seems unnecessary

@evoactivity
Copy link
Copy Markdown
Member

evoactivity commented Nov 30, 2025

Can you run pnpm format please, to get passed the lint task :)

@evoactivity evoactivity added internal Internal changes that still require a mention in the changelog/release notes bug Something isn't working and removed internal Internal changes that still require a mention in the changelog/release notes labels Nov 30, 2025
@evoactivity evoactivity merged commit 4527737 into ember-tooling:main Nov 30, 2025
7 of 8 checks passed
@evoactivity
Copy link
Copy Markdown
Member

evoactivity commented Nov 30, 2025

Thanks for contributing 😄

@github-actions github-actions Bot mentioned this pull request Nov 30, 2025
Merged
@fisker fisker deleted the preprocess branch November 30, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fisker @evoactivity