Skip to content

Add mtproto.zig#221

Merged
jiacai2050 merged 1 commit into
zigcc:mainfrom
sleep3r:add-mtproto-zig
May 17, 2026
Merged

Add mtproto.zig#221
jiacai2050 merged 1 commit into
zigcc:mainfrom
sleep3r:add-mtproto-zig

Conversation

@sleep3r
Copy link
Copy Markdown
Contributor

@sleep3r sleep3r commented May 16, 2026

Summary

  • Add sleep3r/mtproto.zig to the Network section.

Checks

  • make all

@sleep3r sleep3r marked this pull request as ready for review May 16, 2026 14:33
Copilot AI review requested due to automatic review settings May 16, 2026 14:33
@sleep3r sleep3r requested review from jiacai2050 and xihale as code owners May 16, 2026 14:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Zig networking project entry to the awesome list’s Network section.

Changes:

  • Added sleep3r/mtproto.zig as a Telegram MTProto proxy project under Network.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds the sleep3r/mtproto.zig library to the Network section of the README. A review comment suggests repositioning the new entry to maintain alphabetical order by repository owner name for better maintainability.

Comment thread README.md
### Network

- [Nyarum/zigtgshka](https://github.com/Nyarum/zigtgshka) - Memory-safe, high-performance Telegram Bot API library for Zig with zero-cost abstractions and comprehensive examples.
- [sleep3r/mtproto.zig](https://github.com/sleep3r/mtproto.zig) - High-performance Telegram MTProto proxy written in Zig.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The Network section in this repository does not appear to follow a strict alphabetical sorting rule, but new entries are often placed either at the end of the section or grouped by topic. While placing this next to another Telegram-related project is logical, the current insertion at line 421 breaks the loose alphabetical order of the surrounding entries (Nyarum -> sleep3r -> Vexu). However, karlseguin (line 424) and ducdetronquito (line 425) are already out of order. For better maintainability, consider moving this entry to a position that respects alphabetical order by repository owner name, which would be after remeh/statsd-zig (line 433).

@jiacai2050 jiacai2050 merged commit 016db60 into zigcc:main May 17, 2026
5 of 6 checks passed
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.

3 participants