This repository has been archived by the owner on Feb 27, 2021. It is now read-only.
Migrate #2
Open
Migrate #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I've tested this and yes, it *always* requires a FQN. Therefore I removed the extra note regarding namespace/wrapping because even by default Laravel has the `App\` namesapce, which requires this. Fixes #995
… output (#1018) Discovered via #1017 (comment) This only happens if you run an artisan command _before_ `runCommand`, but since every of these tests performs migrations, using artisan, the mocked output instance is left behind and also affected us. The call added in `runCommand` explicitly unregisters `\Illuminate\Console\OutputStyle` and thus also throwing away possibly mocked versions.
* Test dynamic * Try/catch relationships * Update snapshot * Test output
* Bump Laravel versions * Test Laravel 8 * Update composer.json * Update composer.json * normalize composer.json * Tweak MethodTest for Laravel 8 Co-authored-by: barryvdh <[email protected]>
* Make createLocalViewFactory compatible with Laravel 8 Fixes #1024 * tests: move up mockFilesystem so it get be re-used in other tests Note: removed `$this->mockOutput = '';` as it was a no-op, that property doesn't exist (it was probably a typo, but I guess we don't need it anyway) * tests: show basic features for ide-helper:meta working
- build isn't used AFAIK? - add `/` prefix were we know they're only expected to be in the root anyway except `.phpunit.result.cache` which may appear anywhere it's run from
* composer require --dev friendsofphp/php-cs-fixer:^2 * php-cs-fixer: ignore cache and custom local config file * php-cs-fixer: initial config * php-cs-fixer: replace commands in composer * php-cs-fixer: add PSR12 "as good as it currently gets" * php-cs-fixer: apply PSR12 to codebase * tests: add workaround to keep unused imports for snapshot testing * php-cs-fixer: apply no_unused_imports * php-cs-fixer: apply array_syntax short * php-cs-fixer: apply single_quote * php-cs-fixer: switch ordered_imports sort_algorithm to alpha Let's be opinionated here for consistency * php-cs-fixer: split between non-tests and tests and share config * php-cs-fixer: apply declare_strict_types for tests * php-cs-fixer: apply fully_qualified_strict_types * php-cs-fixer: apply space_after_semicolon * php-cs-fixer: apply trailing_comma_in_multiline_array * php-cs-fixer: apply trim_array_spaces * php-cs-fixer: apply unary_operator_spaces * php-cs-fixer: apply whitespace_after_comma_in_array * php-cs-fixer: apply native_function_invocation * php-cs-fixer: apply concat_space * grumphp: reflect we're using phpcsfixer now * composer remove --dev squizlabs/php_codesniffer * gha: simplify fix-style approach Not really necessary to remove packages and then manually prevent unrelated commits creeping in Co-authored-by: Barry vd. Heuvel <[email protected]>
…s` to solve failed build (#1032)
At least until 2.6.6; quite a chore ;) The idea is to keep it updated; ideally each PR updates the changelog otherwise we/I will keep track of the "notable" changes. The goal would be that once the next release is about to happen, the changes collected here so far can just be copy-pasted to the release notes and the top link has to be bumped. I would define "notable" as user-facing, thus many internal changes (readme, test suite, etc.) have not been added to give end-users a clearer focus what may have changed/improved _for them_.
* composer remove --dev phpro/grumphp * Remove remains of grumphp
* New --write-mixin option. * Keep all tags from the existing docblock. * Update argument description to be more descriptive Co-authored-by: Markus Podar <[email protected]> * Add the new --write-mixin option to the docs * Add test for the new --write-mixin option * Update README.md * composer fix-style * Adapt test after refactoring recently done Co-authored-by: Markus Podar <[email protected]>
Co-authored-by: Dominik Krämer <[email protected]>
…cal() (#886) * README.md: $this->app->environment() !== 'production' to $this->app->isLocal() * Renamed to README.md * Remove unrelated changes
* Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * Update run-tests.yml * gha: no need for self-update, setup-php `composer:v2` already gives the version we want Co-authored-by: Markus Podar <[email protected]>
…apshots (#1039) * tests: running in CI should not create but rather fail on missing snapshots Otherwise they get created on the CI infrastructure but that's it, the build won't fail although it was forgotten to add the snapshot. * Bump minimum phpunit version which correctly accepts the `-d` parameter The `prefer-lowest` version we get with L6 is 8.0.0 which throws this error: ``` PHP Fatal error: Uncaught TypeError: ini_set() expects parameter 2 to be string, bool given in /home/runner/work/laravel-ide-helper/laravel-ide-helper/vendor/phpunit/phpunit/src/TextUI/Command.php:379 ``` * Update composer.json Co-authored-by: Barry vd. Heuvel <[email protected]>
* Fix phpdoc generate for custom cast with parameter * Add param for cast * Simplify cast class name normalizing Co-authored-by: Markus Podar <[email protected]> * Update changelog * composer fix-style Co-authored-by: Markus Podar <[email protected]>
* Created a positiblity to add custom relation type * Changed access modifier Co-authored-by: Markus Podar <[email protected]> * fixed hints from code review * Added Test for #987 * composer fix-style * Clarify doc in config entry * Update CHANGELOG.md Co-authored-by: Markus Podar <[email protected]>
* Remove doctrine * Fix tests * normalize composer.json * Add back platform * Bump integration versions * composer fix-style * Fix stability * Bump minimum to 10.36 * Bump minimum to 10.38 * More types * Fast fail off * Bump docblock * composer fix-style * add bpchar * Use match for type detection Co-authored-by: Hafez Divandari <[email protected]> * Use driverName Co-authored-by: Hafez Divandari <[email protected]> * fix match * TWeak types * Update snapshots * Remove testbench 7 * composer fix-style * Update src/Console/ModelsCommand.php Co-authored-by: Hafez Divandari <[email protected]> * Update src/Console/ModelsCommand.php Co-authored-by: Hafez Divandari <[email protected]> * Tweak bool * Update src/Console/ModelsCommand.php Co-authored-by: Hafez Divandari <[email protected]> * composer fix-style * Remove custom types, default to string --------- Co-authored-by: barryvdh <[email protected]> Co-authored-by: laravel-ide-helper <[email protected]> Co-authored-by: Hafez Divandari <[email protected]>
* update dependencies * Implement new methods for `Authenticatable` contract * Added Laravel 11 to CI test target * normalize composer.json * Remove obsolete migration method (column type) from test * Update snapshots * Add changelog entry * Update readme --------- Co-authored-by: 武田 憲太郎 <[email protected]> Co-authored-by: barryvdh <[email protected]>
* Bump minimum PHP version to 8.1 Laravel 10 doesn't support 8.0 anyway * Remove phpunit 9 * gha: remove unused matrix dimension * gha: bump runner os * tests: remove outdated version checks
* Use int/bool instead of longer variant * Add changelog * Update snapshots
* Merge reset/smart reset, don't set class * Add changelog
* Remove `--smart-reset` completely See #1523 (comment) * Update CHANGELOG.md --------- Co-authored-by: Barry vd. Heuvel <[email protected]>
* Prepare 3.0 release * Update CHANGELOG.md * Update CHANGELOG.md
* Add pivot class property * Add test for pivot class properties * Fix failed test for pivot class properties * Update CHANGELOG.md * Fix failed test
* Add `AllowDynamicProperties` Attribute to cooperate with php8.2 deprecation
* lint snapshot
* Fix collection attribute typing
* Update CHANGELOG.md
* Update CHANGELOG.md
* Update CHANGELOG.md
* Update composer.json
* Fix collectino with template types WIP
* Add `AllowDynamicProperties` Attribute to cooperate with php8.2 deprecation
* Fix collection attribute typing
* Update CHANGELOG.md
* Update CHANGELOG.md
* Update composer.json
* composer fix-style
* Fix tests
* Fix tests
* composer fix-style
* Fix collectino with complex template types{
* Delete phpactor
* Install from scrumble repo
* Implemented correct attribute handling
* composer fix-style
* Revert "Install from scrumble repo"
This reverts commit 3f1057a.
* Re-add barryvdh links
* Revert "Update composer.json"
This reverts commit 98252fe.
* Fix pivot test
* Correct CHANGELOG.md with attribute merge request
* More clear PhpDocTypeParser filename and clarified directory according to PSR12
---------
Co-authored-by: GeoSot <[email protected]>
Co-authored-by: laravel-ide-helper <[email protected]>
Co-authored-by: Luukdewaaier <[email protected]>
Co-authored-by: Luuk de Weijer <[email protected]>
* Improve config file - Terminate all descriptions with a dot . - Update outdated descriptions - Add missing examples * Fix capitalizations
* Adding support for casting to AsEnumCollection * Adding missing string to advanced_casts table for enum_collection * Added to snapshots and ArrayCastsWithComment * Update CHANGELOG.md * Reverted automated whitespace, strict changes * Updated test snapshots, tests passing * Whitespace revert --------- Co-authored-by: Barry vd. Heuvel <[email protected]>
Co-authored-by: Barry vd. Heuvel <[email protected]>
* Add `\\static` as check For some reason, the `static` keyword as a return type, does not return `isBuiltin()` === true, so a `\\` is added. We need to manually add it to the built in check. * Update ModelsCommand.php * Add tests * Add CHANGELOG.md entry --------- Co-authored-by: Markus Podar <[email protected]> Co-authored-by: Barry vd. Heuvel <[email protected]>
* set hashed properity as string * add to changelog * add test
* Use the bug-fixed version of the upstream library * Updated the test for fixed features to correctly expect
* feat: support generics eloquent builder * fix: required laravel version * fix: ci execution is only for Laravel-11 * docs: add CHANGELOG * docs: update supported Laravel versions and adjust output examples
* Test against php 8.4 * Test against php 8.4 --------- Co-authored-by: Barry vd. Heuvel <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.