Skip to content

Remove legacy module #63

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
Closed
4 tasks done
tomuben opened this issue Mar 11, 2025 · 0 comments · Fixed by #64
Closed
4 tasks done

Remove legacy module #63

tomuben opened this issue Mar 11, 2025 · 0 comments · Fixed by #64
Assignees
@tomuben
Labels
refactoring Code improvement without behavior change

Comments

@tomuben
Copy link
Contributor

tomuben commented Mar 11, 2025 •
edited
Loading

Summary

Remove legacy module exasol_error_reporting_python. We only use exasol.erro.

Details

All our projects have switched to the new API in exasol.error. We can now remove the legacy module.

Task(s)

  • move all remaing code which is still being used by the new module exasol.error to there
  • remove the module
  • remove the module from the pyproject.toml
  • Update changelog
@tomuben tomuben added the refactoring Code improvement without behavior change label Mar 11, 2025
@tomuben tomuben self-assigned this Mar 11, 2025
tomuben added a commit that referenced this issue Mar 11, 2025
@tomuben
#63: Remove legacy module
e4264f0
@tomuben tomuben mentioned this issue Mar 11, 2025
Merged
tomuben added a commit that referenced this issue Mar 11, 2025
@tomuben
#63: Remove legacy module
67e4638
tomuben added a commit that referenced this issue Mar 11, 2025
@tomuben
Refactoring/63 remove legacy module (#64)
80383a2 
* #63: Remove legacy module

* Updated changelog

* Removed more legacy stuff

* Reformatted _error.py

* Reformatted error_test.py

* Made moved files private

* Fixed _error.py and _placeholder_handler.py

* Fixed unit test
@tomuben tomuben marked this as a duplicate of #59 Mar 11, 2025
@tomuben tomuben mentioned this issue Mar 11, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomuben tomuben

Labels
refactoring Code improvement without behavior change
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactoring/63 remove legacy module exasol/error-reporting-python
1 participant
@tomuben