Skip to content

chore: Use type declaration instead of type hint#7338

Merged
Wirone merged 2 commits intoPHP-CS-Fixer:masterfrom
localheinz:fix/type-declaration
Sep 27, 2023
Merged

chore: Use type declaration instead of type hint#7338
Wirone merged 2 commits intoPHP-CS-Fixer:masterfrom
localheinz:fix/type-declaration

Conversation

@localheinz
Copy link
Copy Markdown
Member

@localheinz localheinz commented Sep 27, 2023

This pull request

  • replaces mentions of type hint and typehint with type declaration

Somewhat related to #7330 (comment).

💁‍♂️ Also see https://www.php.net/manual/en/language.types.declarations.php. A lot of people keep using type hint instead of type declarations, but they are officially called type declarations, so I believe it makes sense to consistently use the official wording. Maybe we can even convince some people to use the official wording.

@localheinz localheinz self-assigned this Sep 27, 2023
@localheinz localheinz marked this pull request as ready for review September 27, 2023 15:31
@localheinz localheinz removed their assignment Sep 27, 2023
@localheinz localheinz marked this pull request as draft September 27, 2023 15:31
@coveralls
Copy link
Copy Markdown

coveralls commented Sep 27, 2023

Coverage Status

coverage: 94.661%. remained the same when pulling 063f85d on localheinz:fix/type-declaration into 2403e0c on PHP-CS-Fixer:master.

@localheinz localheinz self-assigned this Sep 27, 2023
@localheinz localheinz marked this pull request as ready for review September 27, 2023 15:44
@localheinz localheinz assigned localheinz and unassigned localheinz Sep 27, 2023
@localheinz localheinz marked this pull request as draft September 27, 2023 15:52
@localheinz localheinz marked this pull request as ready for review September 27, 2023 16:01
@localheinz localheinz removed their assignment Sep 27, 2023
@Wirone Wirone force-pushed the fix/type-declaration branch from ebb171f to 063f85d Compare September 27, 2023 17:54
@Wirone Wirone changed the title fix: Use type declaration instead of type hint chore: Use type declaration instead of type hint Sep 27, 2023
@Wirone Wirone enabled auto-merge (squash) September 27, 2023 17:56
@Wirone Wirone merged commit 2f4d1a3 into PHP-CS-Fixer:master Sep 27, 2023
@Wirone
Copy link
Copy Markdown
Member

Wirone commented Sep 27, 2023

Thanks @localheinz 🍻!

@localheinz localheinz deleted the fix/type-declaration branch September 27, 2023 18:12
@localheinz
Copy link
Copy Markdown
Member Author

Thank you, @Wirone!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants