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.

Sign up for GitHub

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

Jump to bottom

Update links and other improvements #23

Merged
merged 4 commits into from
May 19, 2024
Merged

Update links and other improvements #23

merged 4 commits into from
May 19, 2024

Conversation

esabol
Copy link
Member

@esabol esabol commented May 18, 2024 •
edited
Loading

This pull request makes the following changes:

  • Replaces "Github" with "GitHub" throughout.
  • Updates the protocol page to match latest PROTOCOL document from gearmand
    GitHub repository.
  • Updates the links for libgearman documentation and such (maybe addresses issue C developers docs should be on website #18).
  • Updates the links in the Perl section. Changes order of Perl modules to put the best maintained module first.

Kind of a "grab bag" of changes, but, whenever I start changing something, I find it difficult to stop!

@esabol esabol mentioned this pull request May 18, 2024
Open
SpamapS
SpamapS approved these changes May 19, 2024
View reviewed changes
Copy link
Member

@SpamapS SpamapS left a comment

Choose a reason for hiding this comment

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

We really should make the protocol docs generated from the PROTOCOL file.. or even more sensible.. make them the same file. :)

@SpamapS SpamapS merged commit e1c6700 into gearman:master May 19, 2024
@SpamapS
Copy link
Member

SpamapS commented May 19, 2024

Also thanks!

@esabol esabol deleted the replace-Github-with-GitHub-throughout branch May 20, 2024 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SpamapS SpamapS SpamapS approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@esabol @SpamapS