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.

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

Refactor endpoint handling #1455

Merged
merged 12 commits into from
Jul 27, 2022
Merged

Refactor endpoint handling #1455

merged 12 commits into from
Jul 27, 2022

Conversation

ardatan
Copy link
Collaborator

@ardatan ardatan commented Jul 24, 2022

Seperate internal plugin for checking if the requested patch matches the endpoint.

@changeset-bot
Copy link

changeset-bot bot commented Jul 24, 2022

⚠️ No Changeset found

Latest commit: 77bb58b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Jul 24, 2022

✅ Benchmark Results

     ✓ no_errors
     ✓ expected_result

     checks.........................: 100.00% ✓ 176586      ✗ 0    
     data_received..................: 24 MB   783 kB/s
     data_sent......................: 10 MB   338 kB/s
     http_req_blocked...............: avg=1.26µs   min=800ns   med=1.1µs   max=323.1µs p(90)=1.6µs   p(95)=1.9µs   
     http_req_connecting............: avg=1ns      min=0s      med=0s      max=139.7µs p(90)=0s      p(95)=0s      
   ✓ http_req_duration..............: avg=258.34µs min=171.6µs med=232.8µs max=20.96ms p(90)=297.6µs p(95)=325µs   
       { expected_response:true }...: avg=258.34µs min=171.6µs med=232.8µs max=20.96ms p(90)=297.6µs p(95)=325µs   
     http_req_failed................: 0.00%   ✓ 0           ✗ 88293
     http_req_receiving.............: avg=20.34µs  min=11.2µs  med=18.5µs  max=7.98ms  p(90)=25.2µs  p(95)=27.5µs  
     http_req_sending...............: avg=8.6µs    min=4.2µs   med=6.1µs   max=3.98ms  p(90)=11.2µs  p(95)=12.6µs  
     http_req_tls_handshaking.......: avg=0s       min=0s      med=0s      max=0s      p(90)=0s      p(95)=0s      
     http_req_waiting...............: avg=229.39µs min=151.4µs med=206.1µs max=20.81ms p(90)=263.1µs p(95)=290.1µs 
     http_reqs......................: 88293   2942.929532/s
     iteration_duration.............: avg=333.01µs min=227.3µs med=303.4µs max=21.56ms p(90)=384.2µs p(95)=417.44µs
     iterations.....................: 88293   2942.929532/s
     vus............................: 0       min=0         max=1  
     vus_max........................: 1       min=1         max=1  

@saihaj
Copy link
Collaborator

saihaj commented Jul 25, 2022

Wait @ardatan can you add a changeset and also some docs for it?

@ardatan
Copy link
Collaborator Author

ardatan commented Jul 26, 2022

I don't think we need a changeset for this because it is a refactor actually.

renovate bot and others added 12 commits July 27, 2022 21:22
* chore(deps): update actions/checkout action to v3 (#1431)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency ioredis to v5.2.2 (#1450)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Replace cross-undici-fetch with @whatwg-node/fetch

* chore(deps): update dependency vite to v3

* Fix GraphiQL build

* Go

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Arda TANRIKULU <ardatanrikulu@gmail.com>
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.

None yet

3 participants