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

revisit patch_applicator error logic #40

Closed
goerlibe opened this issue Nov 10, 2023 · 0 comments · Fixed by #44
Closed

revisit patch_applicator error logic #40

goerlibe opened this issue Nov 10, 2023 · 0 comments · Fixed by #44
Assignees
Labels
priority:high We should deal with this soon refactor Refactoring or Cleanup required

Comments

@goerlibe
Copy link
Collaborator

goerlibe commented Nov 10, 2023

we should check if everything is implemented as intended (and correctly checked in the vs code gui)
discopop-project/discopop#432

@goerlibe goerlibe added priority:medium Important but not urgent refactor Refactoring or Cleanup required labels Nov 10, 2023
@goerlibe goerlibe changed the title revisit patch_generator error logic revisit patch_applicator error logic Nov 10, 2023
@goerlibe goerlibe added priority:high We should deal with this soon and removed priority:medium Important but not urgent labels Nov 22, 2023
@goerlibe goerlibe mentioned this issue Nov 22, 2023
@goerlibe goerlibe self-assigned this Nov 22, 2023
@goerlibe goerlibe linked a pull request Nov 30, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:high We should deal with this soon refactor Refactoring or Cleanup required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant