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

Remove links to ohloh, annocpan and cpanratings #21

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

Conversation

mbeijen
Copy link

@mbeijen mbeijen commented Jan 15, 2022

Those are: not existing anymore, hosting some link farm, and read-only, respectively.

ohloh no longer exists and cpanratings is read only
annocpan.org is now hosting some kind of link farm
Comment on lines -2580 to -2585
=item pod comments and corrections

If you find inaccuracies in the DBD::ODBC pod or have a comment which
you think should be added then go to L<http://annocpan.org> and submit
them there. I get an email for every comment added and will review
each one and apply any changes to the documentation.
Copy link

Choose a reason for hiding this comment

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

Perhaps this should be edited to provide different instructions for the described feedback (github issues, maybe?), rather than simply deleted, leaving no path for that feedback?

Copy link
Author

Choose a reason for hiding this comment

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

I thought about re-writing stating that feedback for POD can be provided via Github. But the github repo is already linked from the module metadata and various other places.

annocpan came from a time where there was no github and collaboration was much more difficult, svn.perl.org n- where this module was hosted previously - was 'read only' for everybody except the author and you had to have a 'commit bit' for pushing and pull requests did not exists, they were diff files.

Of course it might still be desirable to have explicit text soliciting feedback for the POD; and that could definitely added, or the original paragraph can be re-phrased, or whatever. The PR was just for 1. pointing out that the links were invalid, 2. adding a one-click way to remove those invalid links.

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