Skip to content

Make getPathsInPath function more robust, including null handling#4612

Open
JemimaBrewer wants to merge 7 commits into
TeamAmaze:release/4.0from
JemimaBrewer:release/4.0
Open

Make getPathsInPath function more robust, including null handling#4612
JemimaBrewer wants to merge 7 commits into
TeamAmaze:release/4.0from
JemimaBrewer:release/4.0

Conversation

@JemimaBrewer

@JemimaBrewer JemimaBrewer commented Apr 23, 2026

Copy link
Copy Markdown

Description

Make getPathsInPath function more robust, including null handling

Issue tracker

Fixes #4476

Automatic tests

  • Added unit tests for getPathsInPath function

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

@JemimaBrewer JemimaBrewer marked this pull request as draft April 23, 2026 10:28
@JemimaBrewer JemimaBrewer marked this pull request as ready for review April 26, 2026 12:50
@VishnuSanal VishnuSanal added the PR-Awaiting-Initial-Review this PR is awaiting for an initial review label Jun 28, 2026
@VishnuSanal VishnuSanal changed the title Fix getPathsInPath Make getPathsInPath function more robust, including null handling Jun 28, 2026

@VishnuSanal VishnuSanal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

note: I added an empty commit (9f62ecd) to retrigger the ci

Signed-off-by: VishnuSanal <t.v.s10123@gmail.com>
@VishnuSanal VishnuSanal added PR-Awaiting-Final-Review this PR is awaiting a final review/approval and removed PR-Awaiting-Initial-Review this PR is awaiting for an initial review labels Jun 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR-Awaiting-Final-Review this PR is awaiting a final review/approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[getPathsInPath] Plain/whitespace path without scheme triggers splitUri NPE

2 participants