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

feat: export asyncLocalStorage instance to global #266

Merged
merged 2 commits into from
Dec 19, 2023
Merged

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Dec 19, 2023

@fengmk2 fengmk2 requested a review from killagu December 19, 2023 13:41
@fengmk2 fengmk2 changed the base branch from master to 5.x December 19, 2023 13:43
Copy link
Contributor

@killagu killagu left a comment

Choose a reason for hiding this comment

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

LGTM

@fengmk2 fengmk2 marked this pull request as draft December 19, 2023 13:44
@fengmk2 fengmk2 force-pushed the export-als-to-global branch from 1fb382f to 76cb309 Compare December 19, 2023 13:45
@fengmk2 fengmk2 marked this pull request as ready for review December 19, 2023 13:45
Copy link

codecov bot commented Dec 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (5.x@fa19c0d). Click here to learn what that means.

Additional details and impacted files
@@          Coverage Diff           @@
##             5.x     #266   +/-   ##
======================================
  Coverage       ?   99.76%           
======================================
  Files          ?       19           
  Lines          ?     2975           
  Branches       ?      558           
======================================
  Hits           ?     2968           
  Misses         ?        7           
  Partials       ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fengmk2 fengmk2 merged commit 732de10 into 5.x Dec 19, 2023
19 checks passed
@fengmk2 fengmk2 deleted the export-als-to-global branch December 19, 2023 14:17
fengmk2 added a commit that referenced this pull request Dec 19, 2023
fengmk2 added a commit that referenced this pull request Dec 19, 2023
@fengmk2
Copy link
Member Author

fengmk2 commented Dec 19, 2023

封装到 gals 库 #268

fengmk2 added a commit that referenced this pull request Dec 31, 2023
fengmk2 added a commit that referenced this pull request Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

如果使用AsyncLocalStorage,如何能够在不被Egg管理的类中,也能正确地拿到上下文对象
2 participants