Skip to content
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.

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

[5.5] Queue Jobs updates #21298

Closed
wants to merge 4 commits into from
Closed

[5.5] Queue Jobs updates #21298

wants to merge 4 commits into from

Conversation

Artistan
Copy link
Contributor

  • updated to include common methods in the abstract Job class
  • added the required getJobId to the abstract class
    this method is used in all existing Illuminate/Queue/Jobs/* classes
    also allows better code completion via the addition in the Contract.

updated to include common methods in the abstract Job class
added the required getJobId to the abstract class
this method is used in all existing Illuminate/Queue/Jobs/* classes
also allows better code completion via the addition in the Contract.
@tillkruss tillkruss changed the title Queue Jobs updates [5.5] Queue Jobs updates Sep 20, 2017
@taylorotwell
Copy link
Member

We can't add to an abstract class on a point release.

@Artistan Artistan deleted the patch-2 branch September 20, 2017 19:23
Artistan pushed a commit to Artistan/laraframe-docblock that referenced this pull request Oct 4, 2017
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.

2 participants