Skip to content

Update location-display - #19277

Merged
runelite-github-app[bot] merged 5 commits into
runelite:masterfrom
trinhc2:update-location-display
Oct 27, 2026
Merged

runelite-github-app[bot] merged 5 commits into
runelite:masterfrom
trinhc2:update-location-display

Conversation

@trinhc2

@trinhc2 trinhc2 commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Add capitalization config

Add capitalization config
@runelite-github-app

runelite-github-app Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

add build=standard in plugin properties
@riktenx

riktenx commented Oct 26, 2026

Copy link
Copy Markdown
Member

please migrate to filepath per above

@riktenx riktenx added the waiting for author waiting for the pr author to make changes or respond to questions label Oct 26, 2026
@runelite-github-app runelite-github-app Bot added size-s and removed waiting for author waiting for the pr author to make changes or respond to questions size-xs labels Oct 26, 2026
@rsbfox

rsbfox commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

You're still using java.io.* in RegionMap.java and SoundEngine.java

Also this doesn't impact review but you could update your build.gradle to be based on the current one. Yours is from the 2024-10-04 version.

@rsbfox rsbfox added the waiting for author waiting for the pr author to make changes or respond to questions label Oct 26, 2026
@trinhc2

trinhc2 commented Oct 27, 2026

Copy link
Copy Markdown
Contributor Author

hey, what are my options here? is using getResourceAsStream not allowed? i have a json in main/resources that i am trying to access and read

@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Oct 27, 2026
@Alexsuperfly

Copy link
Copy Markdown
Contributor

its because youre constructing a FileNotFoundException

@trinhc2

trinhc2 commented Oct 27, 2026

Copy link
Copy Markdown
Contributor Author

is that the only change needed to pass?

@Alexsuperfly

Alexsuperfly commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

its the only thing currently being flagged to us as a file operation outside of FilePath

@runelite-github-app
runelite-github-app Bot merged commit cfae8fd into runelite:master Oct 27, 2026
2 checks passed
@trinhc2

trinhc2 commented Oct 27, 2026

Copy link
Copy Markdown
Contributor Author

its the only thing currently being flagged to us as a file operation outside of FilePath

thanks for your help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants