Jump to content
Sign in to follow this  
Mystery

Templates : Issues & Pull Request

Recommended Posts

Hey everyone,

 

We recently implemented Issue & Pull Request templates. You can find them here:

 

Basically, when users are creating Issues or Pull Requests, they will see this form. This helps Hercules' Developers & Contributors by having the right details at the start of a thread. It ensures issues get resolved quickly and time isn't wasted. These templates are a step forward to help lessen issues or pull requests that do not meet requirements. Failure to follow these templates will result in an automatic closure of the issue or pull request. 

 

It is important for users to follow these templates and fill them out when creating issues and / or pull requests. 

 

 

Screen Shot 2017-03-11 at 6.36.09 PM.png

Share this post


Link to post
Share on other sites

Yesterday I submit a new issue with new template and it felt like were too much fields to fill for what I was reporting.

 

"Description" and "Current Behaviour" it's like almost the same.

Branch could be assumed to be "master" since other branches are more specific and those may be exceptions to it.

Also an issue related to databases or npcs maybe could not require git hash/revision and leave it as optional, since it is more needed in src issues.

 

I like new templates, but maybe could be a little more flexible when posting issues. If you let me suggest, please bring a field to propose the "kind" of issue (if its related to core, to npcs, to documentation, to database, etc), I think it would help to categorize the issue.

Share this post


Link to post
Share on other sites

Taking into account the care-free nature of most "users" of any RO application, I personally would prefer they mention as much detail as possible. Most definitely the git revision in every report for the slightest of issues.

The raw file will give you a clearer description of the fields in comments while posting PRs or Issues. That said, it may not be as convenient for users as much as it is for the regular reviewers of issues/PRs.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
Sign in to follow this  

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.