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

Allow use of zeitwerk autoloader. #1872

Closed
yulgolem opened this issue Mar 5, 2021 · 0 comments · Fixed by #1873
Closed

Allow use of zeitwerk autoloader. #1872

yulgolem opened this issue Mar 5, 2021 · 0 comments · Fixed by #1873
Projects
.ee registry system

Comments

@yulgolem
Copy link
Contributor

yulgolem commented Mar 5, 2021

Since introducing Rails 6 there are some concurrency issues with standard autoloader, but we got code loading setup in application.rb preventing us from using zeitwerk.

We should fix this setup and migrate to zeitwerk in order to have multithreaded concurrency on delayed jobs.
@yulgolem yulgolem added this to To do in .ee registry system via automation Mar 5, 2021
@yulgolem yulgolem mentioned this issue Mar 5, 2021
Merged
@yulgolem yulgolem moved this from To do to Reviewer approved in .ee registry system Mar 8, 2021
.ee registry system automation moved this from Reviewer approved to Done Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
.ee registry system
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Introduce Zeitwerk class loader internetee/registry
1 participant
@yulgolem