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

Add request handling #1682

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Add request handling #1682

merged 1 commit into from
Nov 7, 2024

Conversation

Draggu
Copy link
Contributor

@Draggu Draggu commented Oct 24, 2024 •
edited
Loading

Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This was referenced Oct 24, 2024
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
mkaput
mkaput requested changes Oct 25, 2024
View reviewed changes
scarb/src/ops/proc_macro_server/mod.rs Outdated Show resolved Hide resolved
scarb/src/ops/proc_macro_server/json_rpc.rs Outdated Show resolved Hide resolved
scarb/src/ops/proc_macro_server/mod.rs Show resolved Hide resolved
mkaput
mkaput requested changes Oct 25, 2024
View reviewed changes
scarb/src/ops/proc_macro_server/mod.rs Outdated Show resolved Hide resolved
@Draggu Draggu force-pushed the spr/main/723491f5 branch from b782912 to d61cf7e Compare November 4, 2024 12:27
@Draggu Draggu force-pushed the spr/main/2ceebe3f branch from 00d9465 to 866da3a Compare November 4, 2024 12:27
@Draggu Draggu changed the base branch from spr/main/2ceebe3f to main November 4, 2024 14:11
@Draggu Draggu mentioned this pull request Nov 4, 2024
Merged
@Draggu Draggu force-pushed the spr/main/723491f5 branch from d61cf7e to 650d221 Compare November 4, 2024 14:11
@Draggu Draggu mentioned this pull request Nov 4, 2024
Closed
@Draggu Draggu changed the base branch from main to spr/main/2ceebe3f November 4, 2024 14:11
@Draggu Draggu force-pushed the spr/main/723491f5 branch from 650d221 to fc3d5a0 Compare November 4, 2024 14:18
@Draggu Draggu force-pushed the spr/main/2ceebe3f branch 2 times, most recently from de2122f to f256d8a Compare November 4, 2024 15:22
@Draggu Draggu force-pushed the spr/main/723491f5 branch 2 times, most recently from 4abbf99 to 8ddf509 Compare November 5, 2024 15:17
@Draggu Draggu force-pushed the spr/main/2ceebe3f branch from f256d8a to 58c0e91 Compare November 5, 2024 15:17
@Draggu Draggu changed the base branch from spr/main/2ceebe3f to main November 5, 2024 15:28
@Draggu Draggu changed the base branch from main to spr/main/2ceebe3f November 5, 2024 15:28
@Draggu Draggu mentioned this pull request Nov 5, 2024
Merged
@Draggu Draggu force-pushed the spr/main/2ceebe3f branch from 58c0e91 to dbec1c1 Compare November 6, 2024 10:40
@Draggu Draggu force-pushed the spr/main/723491f5 branch from 8ddf509 to 897fb6d Compare November 6, 2024 10:40
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2024
@Draggu
Add derives to cairo-lang-macro (#1676)
b7caa82 
**Stack**:
- #1686
- #1685
- #1684
- #1683
- #1718
- #1682
- #1681
- #1680
- #1679
- #1715
- #1676 ⬅


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2024
@Draggu
Proc macro server API (#1715)
2b3137b 
**Stack**:
- #1686
- #1685
- #1684
- #1683
- #1718
- #1682
- #1681
- #1680
- #1679
- #1715 ⬅
- #1676


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2024
@Draggu
Add proc-macro-server command (#1679)
a00fe3f 
**Stack**:
- #1686
- #1685
- #1684
- #1683
- #1718
- #1682
- #1681
- #1680
- #1679 ⬅
- #1715
- #1676


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@Draggu Draggu force-pushed the spr/main/723491f5 branch from 897fb6d to 996f9f7 Compare November 6, 2024 17:54
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2024
@Draggu
Proc macro server: Load ProcMacroHost (#1680)
ccdf19a 
**Stack**:
- #1686
- #1685
- #1684
- #1683
- #1718
- #1682
- #1681
- #1680 ⬅


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
@Draggu Draggu force-pushed the spr/main/723491f5 branch from 996f9f7 to 4644a0a Compare November 7, 2024 10:08
github-merge-queue bot pushed a commit that referenced this pull request Nov 7, 2024
@Draggu
Add Proc macro server (#1681)
8297e78 
**Stack**:
- #1686
- #1685
- #1684
- #1683
- #1718
- #1682
- #1681 ⬅


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*
Base automatically changed from spr/main/2ceebe3f to main November 7, 2024 11:53
@Draggu
Add request handling
f49b44d 
commit-id:723491f5
@Draggu Draggu force-pushed the spr/main/723491f5 branch from 4644a0a to f49b44d Compare November 7, 2024 12:45
@Draggu Draggu enabled auto-merge November 7, 2024 12:46
@Draggu Draggu added this pull request to the merge queue Nov 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Nov 7, 2024
@Draggu Draggu added this pull request to the merge queue Nov 7, 2024
Merged via the queue into main with commit d58f2fd Nov 7, 2024
22 checks passed
@Draggu Draggu deleted the spr/main/723491f5 branch November 7, 2024 16:25
@BrewTestBot BrewTestBot mentioned this pull request Nov 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@piotmag769 piotmag769 piotmag769 approved these changes

@mkaput mkaput mkaput approved these changes

@maciektr maciektr Awaiting requested review from maciektr

@DelevoXDG DelevoXDG Awaiting requested review from DelevoXDG

@wawel37 wawel37 Awaiting requested review from wawel37

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

Successfully merging this pull request may close these issues.
3 participants
@Draggu @mkaput @piotmag769