Skip to content

Add QUERY http method support - #82

Open
ivamly wants to merge 1 commit into
gleam-lang:v5from
ivamly:feat/query
Open

ivamly wants to merge 1 commit into
gleam-lang:v5from
ivamly:feat/query

Conversation

@ivamly

@ivamly ivamly commented Jul 7, 2026 •

Copy link
Copy Markdown

Add QUERY HTTP method support

Closes #81

@ivamly

ivamly commented Jul 7, 2026

Copy link
Copy Markdown
Author

Hey! This is my first PR in anything Gleam-related and I'm super excited about it!
Happy to make any changes you need 🙌

@lpil
lpil changed the base branch from main to v5 September 6, 2026 10:32
@lpil

lpil commented Sep 6, 2026

Copy link
Copy Markdown
Member

Hello! I've changed this to target the v5 branch. Could you remove the formatting changes and rebase to remove the merge conflicts please 🙏

@lpil

lpil commented Sep 7, 2026

Copy link
Copy Markdown
Member

Tag me once you're done and need a review please @ivamly. Thank you!

@lpil
lpil marked this pull request as draft September 7, 2026 09:21
@ivamly
ivamly force-pushed the feat/query branch 3 times, most recently from cba1ac5 to 2b2f4d9 Compare September 14, 2026 19:09
@ivamly
ivamly marked this pull request as ready for review September 14, 2026 19:10
@ivamly

ivamly commented Sep 14, 2026

Copy link
Copy Markdown
Author

@lpil done!

@ivamly

ivamly commented Sep 21, 2026

Copy link
Copy Markdown
Author

@lpil ping

@lpil lpil left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you. One note inline!

Comment thread src/gleam/http.gleam
/// Proxies that support this should only do so for known trustworthy
/// clients.
Connect
Query

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Include documentation for this new method please, matching the others 🙏

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done

@ivamly
ivamly force-pushed the feat/query branch 2 times, most recently from 0f327c2 to 41714e8 Compare September 22, 2026 18:06
@ivamly

ivamly commented Sep 22, 2026 •

Copy link
Copy Markdown
Author

@lpil, fixed

@ivamly

ivamly commented Sep 28, 2026

Copy link
Copy Markdown
Author

@lpil ping

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.

feat: Add QUERY method (RFC 10008) support

2 participants