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

Patch NEU Repo Parser bugs #979

Closed
wants to merge 1 commit into from
Closed

Patch NEU Repo Parser bugs #979

wants to merge 1 commit into from

Conversation

AzureAaron
Copy link
Collaborator

  • Close readers once we're done with them to avoid fatal resource leaks
  • Make item loading atomic

@AzureAaron
Patch NEU Repo Parser bugs
1b0e98e 
- Close readers once we're done with them to avoid fatal resource leaks
- Make item loading atomic
@AzureAaron AzureAaron added bug Something isn't working crash This is a crash small change Small changes that are easier to review labels Sep 1, 2024
@AzureAaron AzureAaron added this to the 1.22.2 milestone Sep 1, 2024
@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Sep 1, 2024
@AzureAaron AzureAaron added the merge conflicts This PR has merge conflicts that need solving. label Sep 2, 2024
@LifeIsAParadox
Copy link
Member

vic solved the problem #982

@LifeIsAParadox LifeIsAParadox removed the reviews needed This PR needs reviews label Sep 2, 2024
@AzureAaron AzureAaron deleted the neu-repo-parser-patches branch September 3, 2024 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working crash This is a crash merge conflicts This PR has merge conflicts that need solving. small change Small changes that are easier to review
Projects
None yet
Milestone
1.22.2
Development

Successfully merging this pull request may close these issues.
2 participants
@AzureAaron @LifeIsAParadox