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

geanyctags, projectorganizer, vimode: Some README and URL updates #1348

Merged
merged 3 commits into from
May 19, 2024

Conversation

techee
Copy link
Member

@techee techee commented May 5, 2024

  1. Suggest using Github for reporting bugs and pull requests instead of sending them by email.
  2. Use https:// instead of http://
  3. Mention universal-ctags as a maintained version of ctags in geanyctags README

Copy link
Member

@elextr elextr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of suggestions, otherwise looks good.

@@ -11,7 +11,9 @@ GeanyCtags adds a simple support for generating and querying ctags files for a G
project. It requires that the ctags command is installed in a system path. On
unix systems, distributions usually provide the ctags package; on Windows, the
ctags binary can be found in the zip Windows distribution from the ctags home
page (http://ctags.sourceforge.net).
page (https://ctags.sourceforge.net). There is also a new, maintained implementation
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given the last release of ectags was 2009 it seems better to not mention it. Uctags has 15 years of improvement and makes Windows and mac binaries.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I've updated this section. I kept the information about ctags though so people know what the difference between ctags and universal ctags is (I think distributions still package the unmaintained ctags).

Please direct all questions, bug reports and patches to the combined geany-plugins
project at https://github.com/geany/geany-plugins and open the corresponding
bug report or pull request there. To notify the author of this plugin about
your post, mention him using his github user name (@techee).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe mention that the github username can usually be found in the MAINTAINERS file.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll probably skip this level of indirection and mention it directly in the documentation. I normally check geany-plugins only from time to time but I have an email notification enabled when someone mentions me.

@techee techee merged commit 51ed8e4 into geany:master May 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants