Skip to content

Update for PM 3.14.0#11

Open
Xenophilicy wants to merge 5 commits intojojoe77777:masterfrom
xenoservers:master
Open

Update for PM 3.14.0#11
Xenophilicy wants to merge 5 commits intojojoe77777:masterfrom
xenoservers:master

Conversation

@Xenophilicy
Copy link
Copy Markdown

Fixes lots of old, outdated API calls. Added support for using FormAPI virion rather than depending on the plugin. Fixed form images crashing issue.

Fixes lots of old, outdated API calls. Added support for using FormAPI virion rather than depending on the plugin. Fixed form images crashing issue.
@dktapps
Copy link
Copy Markdown
Collaborator

dktapps commented Jun 27, 2020

This makes a bunch of formatting and doc changes that have nothing to do with the functionality.

@Xenophilicy
Copy link
Copy Markdown
Author

Is there a reason those PHPDoc comments and formatting changes should be reverted?

@dktapps
Copy link
Copy Markdown
Collaborator

dktapps commented Jun 27, 2020

because they

have nothing to do with the functionality.

Keep your changes separate.

@Xenophilicy
Copy link
Copy Markdown
Author

So, do you just want all the PHPDoc comments removed or something? I can just add "Updated PHPDoc comments" to the PR description if that makes you happy. That way the PR will be accurate to your liking and it won't just be a "functionality" update.

@dktapps
Copy link
Copy Markdown
Collaborator

dktapps commented Jun 28, 2020

It adds a bunch of noise to the diff that doesn't contribute towards the intended purpose (updating to latest PM), as such it should be done separately. I wouldn't accept this PR if it were to PM unless the changes were separated. Not to mention the fact that the doc comments I can see are just useless auto generated garbage anyway.

@Xenophilicy
Copy link
Copy Markdown
Author

@dktapps Ready for re-review of changes. All the "auto generated garbage" and "formatting and doc changes" should be undone now lol 👍

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