Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Clients link fixed in README #16569

Merged

Conversation

niranjan-kurhade
Copy link
Contributor

In this PR, I have fixed the broken link of clients in the README file. Now it redirects to the correct link.

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Pull request includes a sign off
  • Code style is correct
    (run the linters)

@niranjan-kurhade niranjan-kurhade requested a review from a team as a code owner October 28, 2023 20:43
@niranjan-kurhade
Copy link
Contributor Author

Hey @DMRobertson @clokep Can you please review my PR?

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

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

Hey @DMRobertson @clokep Can you please review my PR?

PRs are automatically put into a queue and reviewed when we can get to them. You're unlikely to get an immediate response on a weekend, however.

This seems sane to me. Please add a changelog entry and sign-off per the PR template.

Signed-off-by: Niranjan Kurhade <[email protected]>
@niranjan-kurhade
Copy link
Contributor Author

Hey @DMRobertson @clokep Can you please review my PR?

PRs are automatically put into a queue and reviewed when we can get to them. You're unlikely to get an immediate response on a weekend, however.

This seems sane to me. Please add a changelog entry and sign-off per the PR template.

Hey @DMRobertson Sorry for tagging, I have now added my Changelog file and signed off as well.

Copy link
Contributor

@DMRobertson DMRobertson left a comment

Choose a reason for hiding this comment

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

Thank you!

@DMRobertson DMRobertson enabled auto-merge (squash) October 30, 2023 16:04
@DMRobertson DMRobertson merged commit 91aa52c into matrix-org:develop Oct 30, 2023
29 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken link (client breakdown) in README
2 participants