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.

Sign up for GitHub

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

Encode URI components in buildURL function #1824

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Merge branch 'main' into mboulais/encode-uri-components-in-build-url

9a1df7d
Select commit
Loading
Failed to load commit list.
Open

Encode URI components in buildURL function #1824

Merge branch 'main' into mboulais/encode-uri-components-in-build-url
9a1df7d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 17, 2025 in 0s

28.57% of diff hit (target 44.00%)

View this Pull Request on Codecov

28.57% of diff hit (target 44.00%)

Annotations

Check warning on line 99 in lib/public/utilities/fetch/buildUrl.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/utilities/fetch/buildUrl.js#L99 

Added line #L99 was not covered by tests

Check warning on line 122 in lib/public/utilities/fetch/buildUrl.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/utilities/fetch/buildUrl.js#L122 

Added line #L122 was not covered by tests

Check warning on line 127 in lib/public/utilities/fetch/buildUrl.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/utilities/fetch/buildUrl.js#L127 

Added line #L127 was not covered by tests

Check warning on line 131 in lib/public/utilities/fetch/buildUrl.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/utilities/fetch/buildUrl.js#L131 

Added line #L131 was not covered by tests

Check warning on line 57 in lib/public/views/Environments/Overview/EnvironmentOverviewModel.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Environments/Overview/EnvironmentOverviewModel.js#L56-L57 

Added lines #L56 - L57 were not covered by tests

Check warning on line 94 in lib/public/views/Logs/Overview/LogsOverviewModel.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Logs/Overview/LogsOverviewModel.js#L94 

Added line #L94 was not covered by tests

Check warning on line 111 in lib/public/views/Runs/Overview/RunsOverviewModel.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Runs/Overview/RunsOverviewModel.js#L111 

Added line #L111 was not covered by tests

Check warning on line 276 in lib/public/views/Tags/Details/TagDetailsModel.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Tags/Details/TagDetailsModel.js#L276 

Added line #L276 was not covered by tests

Check warning on line 63 in lib/public/views/Tags/Overview/TagsOverviewModel.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Tags/Overview/TagsOverviewModel.js#L63 

Added line #L63 was not covered by tests
View more details on Codecov