Convert .range to .location

rokucommunity / brighterscript

A superset of Roku's BrightScript language

MIT License

152 stars 47 forks source link

Convert .range to .location #1231

Closed TwitchBronBron closed 3 weeks ago

TwitchBronBron commented 3 weeks ago

Convert all .range properties to .location. This allows all tokens to know what file they came from. This is a fairly significant breaking change.

TODO

[x] convert all .range to .location
[x] add unit tests ensuring sourcemaps properly track tokens from multiple files
[x] ensure transpiled output is the same for a few large projects

rokucommunity-bot[bot] commented 3 weeks ago

Hey there! I just built a new temporary npm package based on e1a9c52256a3ae3765c1d546693f6b5caa951fe3. You can download it here or install it by running the following command:

npm install https://github.com/rokucommunity/brighterscript/releases/download/v0.0.0-packages/brighterscript-1.0.0-alpha.32-location-tracking-with-uri.20240613195621.tgz

rokucommunity-bot[bot] commented 3 weeks ago

Hey there! I just built a new temporary npm package based on ee1f22026799f6275245ed845ea0c838e76c9323. You can download it here or install it by running the following command:

npm install https://github.com/rokucommunity/brighterscript/releases/download/v0.0.0-packages/brighterscript-1.0.0-alpha.32-location-tracking-with-uri.20240614135552.tgz

rokucommunity-bot[bot] commented 3 weeks ago

Hey there! I just built a new temporary npm package based on 91ff76e090b53667c89588dc0dd70f83c1fa22c9. You can download it here or install it by running the following command:

npm install https://github.com/rokucommunity/brighterscript/releases/download/v0.0.0-packages/brighterscript-1.0.0-alpha.32-location-tracking-with-uri.20240614171847.tgz

TwitchBronBron commented 3 weeks ago

I finished testing on a few projects, everything seems to be working properly. I'm going to merge this, and hopefully the next round of alpha testing will flush out any unexpected issues.