From Idea to Merged Laravel Pull Request ✅

Поделиться
HTML-код
  • Опубликовано: 25 окт 2024

Комментарии • 22

  • @elijahcruz8895
    @elijahcruz8895 2 месяца назад +9

    I absolutely love the CLI improvements that you guys have been adding! Especially now that we have "make:view", and adding the flags have been absolutely amazing!

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +2

      We love them too. Thank you 🙏 (Christoph)

    • @ajzack983
      @ajzack983 2 месяца назад

      @@LaravelPHP I hope we can add migration cols from the cli as well

  • @rcknr
    @rcknr 2 месяца назад

    I have made a PR to Valet which has been due for a review since January... )))

  • @NathanBudd
    @NathanBudd 2 месяца назад +1

    This is cool, but I'd imagine anyone watching this would have been in the same scenario I was when I attempted my first PR, and that was that I had to rebase my code.
    If you're a solo dev watching this (not part of a team), rebasing could well be foreign to you. Might be a good idea to explain this in a future video?

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +1

      thanks for mentioning. This is important indeed. will give it some thoughts. (Christoph)

  • @smetzdev
    @smetzdev 2 месяца назад

    Hey Christoph, any tips on how to setup "actions on save" to follow the code-style guideline from laravels contribution page? Thank you for your great videos!

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +2

      Hey 👋 I believe it should be possible with Pint "on save", but I haven't tried it. This might help: ruclips.net/video/uDipoNN7fhQ/видео.html (Christoph)

    • @smetzdev
      @smetzdev 2 месяца назад

      @@LaravelPHP Thank you!

  • @abrhamb2952
    @abrhamb2952 Месяц назад

    Thank you. I wish this video how to create `Artisan::block(['create:super-admin', 'delete:all-file','download:this-url']);`

  • @emredikmen276
    @emredikmen276 2 месяца назад

    It was an informative content, thank you.

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +1

      Glad it was helpful! (Christoph)

  • @andreich1980
    @andreich1980 2 месяца назад +1

    PhpStorm has an option to collapse unchanged code when viewing diff

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +2

      Good to know, thanks (Christoph)

  • @fasterisq2.1m76
    @fasterisq2.1m76 2 месяца назад

    Good

  • @dominikbetlej7114
    @dominikbetlej7114 2 месяца назад

    Thanks ;d

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +1

      You're welcome 👍

  • @samubella
    @samubella 2 месяца назад

    It would be cool to have a default value for the view name based on the class name, is that possible?
    default: Str::kebab($name)

    • @LaravelPHP
      @LaravelPHP  2 месяца назад +2

      Sounds good, but I guess we would need this then for all other commands to to keep it consitent. (Christoph)