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

Jump to bottom

test: replace `$request->uri` with `$request->getUri()` #5574

Merged

kenjis merged 1 commit into codeigniter4:develop from kenjis:fix-request-uri

Jan 15, 2022

Member

kenjis commented Jan 15, 2022 •

edited

Loading

Description
Follow-up #5346

refactor: $request->uri → $request->getUri()

Checklist:

Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
Conforms to style guide


          test: refactor: $request->uri → $request->getUri()

015b302

kenjis added the testing label

kenjis changed the title ~~test: refactor: $request->uri → $request->getUri()~~ test: replace $request->uri with $request->getUri()

paulbalandan approved these changes

View reviewed changes

kenjis merged commit 034a965 into codeigniter4:develop

kenjis deleted the fix-request-uri branch

January 15, 2022 09:52

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

Labels

testing