Skip to content

Feature: Publish stubs #2978

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
Merged
merged 2 commits into from
Mar 25, 2025
Merged

Feature: Publish stubs #2978

merged 2 commits into from
Mar 25, 2025

Conversation

xav-developer
Copy link
Contributor

Fixes #

Proposed Changes

  • Added command : php artisan orchid:stub-publish

After:

After executing the publishes in the structure, the modified stubs are picked up during generation.

stubs
└── orchid
    └── platform
        ├── chart.stub
        ├── filters.stub
        ├── listener.stub
        ├── presenter.stub
        ├── rows.stub
        ├── screen.stub
        ├── selection.stub
        ├── table.stub
        └── tabMenu.stub

@tabuna
Copy link
Member

tabuna commented Mar 24, 2025

I think publishing stubs is a very useful practice, and I really like it! 😊

By the way, the command php artisan orchid:stub-publish feels a bit long and is in singular form.
Maybe we could make it shorter and more intuitive, like php artisan orchid:stubs? 🤔

@xav-developer
Copy link
Contributor Author

@tabuna

Renamed command to: php artisan orchid:stubs

@tabuna tabuna merged commit ca7ae51 into orchidsoftware:master Mar 25, 2025
44 of 46 checks passed
tabuna added a commit that referenced this pull request Mar 26, 2025
@tabuna
refs #2978 Improve published stubs
ea6d6ef
tabuna added a commit that referenced this pull request Mar 27, 2025
@tabuna
refs #2978 Added tests for publish stubs
4393ff5
tabuna added a commit that referenced this pull request Mar 27, 2025
@tabuna
refs #2978 Improve published stubs
8192063
tabuna added a commit that referenced this pull request Mar 27, 2025
@tabuna
refs #2978 Added tests for publish stubs
04bda76
tabuna added a commit that referenced this pull request Mar 27, 2025
@tabuna
refs #2978 Improve published stubs (#2982)
a846de1 
* refs #2978 Improve published stubs
* refs #2978 Added tests for publish stubs
---------

Co-authored-by: tabuna <5102591+tabuna@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xav-developer @tabuna