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

Update README.md #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #44

wants to merge 1 commit into from

Conversation

MagicFab
Copy link

Added compatibility section and Debian 10 info

Added compatibility section and Debian 10 info
@ishikawa999
Copy link
Collaborator

@MagicFab

Thank you for sharing the results of the operation test.

I thought it would be better not to merge this pull request for the following reasons:
In this Redmine theme, the corresponding Redmine version is different for each branch. Install section
For example, the code in the master branch is not compatible with 4.0.3-stable.
People who see the "Compatibility section" you've added may mistakenly think that the master branch is compatible with 4.0.3-stable.

Also, Debian information is probably unnecessary as the theme is OS independent.

@MagicFab
Copy link
Author

I think it's still valuable to save time to others. The instructions are clear which branch to clone when using different versions, that
s why I included the specific Debian version too. Redmine is not packaged yet in the current Debian stable version (11) so the Debian version was important too.

In the end the point is your theme has specific support for a full stable Debian + Redmine (cleanly packaged and supported), which I haven't seen elsewhere.

@MagicFab
Copy link
Author

Many thanks for providing specific versions for older Redmine versions :) If there is a way to donate or support this theme otherwise I would love to do that.

@ishikawa999
Copy link
Collaborator

ishikawa999 commented Sep 3, 2021

@MagicFab
Thank you for your response.

I think it's still valuable to save time to others.

I'm sure you're right.
The Redmine version corresponding to the install section is listed, but it seems kinder to have the Compatibility section as well.

I'll try to merge this pull request and add further explanation for other versions.
Example:

I have confirmed that the theme works in the next version.(As of 2021/8/31)

* Redmine3.4.x and 3.4 branch theme for this repository
* Redmine4.0.x and 4.0 branch theme for this repository

I can't work now, so I'll do it later.

s why I included the specific Debian version too. Redmine is not packaged yet in the current Debian stable version (11) so the Debian version was important too.

Yes, it is useful information that Redmine 4.0 runs on Debian.
However, I don't think this information is necessary to run this theme.
In general, installing a theme is something you think about after you have installed Redmine, and you don't read the README of a theme when you think about whether Redmine can be used on your OS.
If you want to know how to run Redmine, you should read the README of Redmine itself, its documentation, or your own personal articles describing the installation procedure of Redmine.

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