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

Convert .range to .location #1231

Merged
merged 10 commits into from
Jun 14, 2024
Merged

Conversation

TwitchBronBron
Copy link
Member

@TwitchBronBron TwitchBronBron commented Jun 13, 2024

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

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

@TwitchBronBron TwitchBronBron added this to the v1.0.0 milestone Jun 13, 2024
@TwitchBronBron TwitchBronBron added the create-package create a temporary npm package on every commit label Jun 13, 2024
@rokucommunity-bot
Copy link

Hey there! I just built a new temporary npm package based on e1a9c52. 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
Copy link

Hey there! I just built a new temporary npm package based on ee1f220. 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
Copy link

Hey there! I just built a new temporary npm package based on 91ff76e. 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
Copy link
Member Author

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.

@TwitchBronBron TwitchBronBron merged commit 650bac5 into release-1.0.0 Jun 14, 2024
6 checks passed
@TwitchBronBron TwitchBronBron deleted the location-tracking-with-uri branch June 14, 2024 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-package create a temporary npm package on every commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants