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

Fix phpdoc in FeatureEvent #42585

Closed
wants to merge 2 commits into from
Closed

Fix phpdoc in FeatureEvent #42585

wants to merge 2 commits into from

Conversation

Denitz
Copy link
Contributor

@Denitz Denitz commented Jan 1, 2024

Summary of Changes

Invalid phpdoc

Testing Instructions

Feature/unfeature article.

Actual result BEFORE applying this Pull Request

Invalid phpdoc

Expected result AFTER applying this Pull Request

Valid phpdoc

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@Denitz Denitz changed the title Fix phpdoc in Update FeatureEvent Fix phpdoc in FeatureEvent Jan 1, 2024
@richard67
Copy link
Member

@Denitz As far as I can see, the same issue already exists in the 4.,4-dev branch. So if we consider your fix to be a bug fix, it should be made for 4.4-dev and later will be merged up into 5.0-dev. And if we consider it a new feature (which would be not appropriate, I think) it would go into 5.1-dev. I've asked maintainers for their opinion and will let you know the result, but I think it would also be ok if you just rebase to or redo it for 4.4-dev.

@Denitz
Copy link
Contributor Author

Denitz commented Jan 2, 2024

@richard67 Yes, you are right, 4.4-dev is the right place for this:
#42587

@Denitz Denitz closed this Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants