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

new features #53

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

new features #53

wants to merge 8 commits into from

Conversation

Bruno86
Copy link

@Bruno86 Bruno86 commented Feb 23, 2023

Hello I added some new features :

  • Added support to add in @author as a signature for a class
  • Added an option to align the name of parameters of a method

I also merged this other fork : https://github.com/Samuel-Hinchliffe/vscode-phpdoc-generator.git

@ronvanderheijden
Copy link
Owner

ronvanderheijden commented Feb 24, 2023

Thank you for submitting your pull request to my repository. I appreciate your effort in contributing to my project. However, after reviewing your pull request, I regret to inform you that I cannot accept it at this state.

I have found that the quality of your work does not meet my standards. Below some highlights:

  • I found some links and README.md files that are incorrectly changed.
  • I found personal information in some examples.
  • It is not legal to change the author of a forked GitHub repository without the original author's permission. The original author of a repository holds the copyright to the code they have created, and changing the authorship of the code without their consent may infringe on their rights.

Please do not take this rejection as a reflection of your abilities. I understand that you have put in time and effort into creating this pull request, and I appreciate your contribution to my project.

I hope to see your future contributions to my project.

Thank you,

Ron van der Heijden

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.

4 participants