Project

General

Profile

Actions

Patch #28996

closed

Update Redmine::Plugin documentation

Added by Sho HASHIMOTO over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Documentation
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

I want to more information to Redmine methods for plugin development.
As a trial, I created a patch to add below information to Redmine::Plugin.

  • Add Marc Since to each Redmine::Plugin methods
  • Add reference to Plugin_Tutorial wiki and Redmine::Plugin#requires_redmine
  • Add description to a few methods

I checked Marc Since info with using yard diff(the result pasted below).

I will create patches to add Marc Since information to Redmine methods after this tickets was closed.


Files

update_redmine_plugin.patch (12.3 KB) update_redmine_plugin.patch Sho HASHIMOTO, 2018-06-08 16:51
Actions #1

Updated by Mizuki ISHIKAWA over 6 years ago

+1
I think this is necessary information for plugin developers.

Actions #2

Updated by Go MAEDA over 6 years ago

  • Assignee set to Jean-Philippe Lang
  • Target version set to 4.0.0

I think the comment suggested by Sho is useful. However, I wonder if we can continuously update the comments in the future versions.

I would like Jean-Philippe to decide whether we can commit this change.

Actions #3

Updated by Sho HASHIMOTO over 6 years ago

Please remove Marc Since tags from patch if it is difficult to maintain version info. I'll publish unofficial document.

Actions #4

Updated by Sho HASHIMOTO about 6 years ago

I'll publish unofficial document.

Like below. This is auto generated table by using yard diff version1 version2 under redmine repository.

Actions #5

Updated by Jean-Philippe Lang about 6 years ago

  • Subject changed from Update Redmine::Plugin document to Update Redmine::Plugin documentation
  • Status changed from New to Closed

It's perfectly right to improve the documentation. I've committed the proposed additions with slight changes, thanks. But the yard tags are not used anywhere in the code, it doesn't sound so good to me to introduce them in a single class.

Actions #6

Updated by Sho HASHIMOTO about 6 years ago

Thanks to merge documentation!!

But the yard tags are not used anywhere in the code, it doesn't sound so good to me to introduce them in a single class.

No. Redmine already has yard tags.

application_helper.rb has yardoc tag because rdoc has no @<tagname>. See below.

Actions

Also available in: Atom PDF