Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Expose process_index context variable for server processes #31

Merged
merged 3 commits into from
Jan 10, 2022

Conversation

achimnol
Copy link
Owner

This PR adds a context variable process_idex to the aiotools.server module so that the worker processes could read the index of current process anywhere.

@achimnol achimnol self-assigned this Jan 10, 2022
@codecov
Copy link

codecov bot commented Jan 10, 2022

Codecov Report

Merging #31 (1213aa3) into main (3a65b38) will decrease coverage by 0.12%.
The diff coverage is 71.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   77.38%   77.26%   -0.13%     
==========================================
  Files          10       10              
  Lines         849      862      +13     
==========================================
+ Hits          657      666       +9     
- Misses        192      196       +4     
Impacted Files Coverage Δ
src/aiotools/server.py 50.17% <71.42%> (+0.90%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a65b38...1213aa3. Read the comment docs.

@achimnol achimnol merged commit 25cfa24 into main Jan 10, 2022
@achimnol achimnol deleted the feature/expose-pidx-as-global-contextvar branch January 10, 2022 09:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant