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

Merge v2 into v3 #501

Merged
merged 11 commits into from
Nov 17, 2023
Merged

Merge v2 into v3 #501

merged 11 commits into from
Nov 17, 2023

Conversation

FWDekker
Copy link
Owner

Fixes #489.

Doesn't actually contain any code changes. This is purely symbolic so I can tell myself v2 is finally over.

FWDekker and others added 11 commits August 2, 2021 19:38
The additional information was introduced by accident in #376 when implementing #373 to add a timeout for generating values. Error messages in the preview (and other places) would contain the full name of the exception that caused the error, which is completely unnecessary.

This fix was originally implemented in the `uds` branch, but I decided to add a "backports" branch for fixes I might want to release in a v2.x.x release before releasing v3.0.0.
Brings the v2 branch up-to-speed with updates from v3. Also adds my full email address and resolves some style errors. Basically a big deserved maintenance edit.
And fix some smaller issues in code here and there, which have already been fixed in the v3 branch.
@FWDekker FWDekker added the code quality Code changes without behavior changes label Nov 17, 2023
@FWDekker FWDekker added this to the v3.0.0 milestone Nov 17, 2023
@FWDekker FWDekker self-assigned this Nov 17, 2023
Copy link

codecov bot commented Nov 17, 2023

Codecov Report

Merging #501 (e4c4382) into main (96a0856) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #501   +/-   ##
=======================================
  Coverage   85.04%   85.04%           
=======================================
  Files          52       52           
  Lines        1993     1993           
  Branches      335      335           
=======================================
  Hits         1695     1695           
  Misses        235      235           
  Partials       63       63           

@FWDekker FWDekker merged commit 5684c54 into main Nov 17, 2023
2 checks passed
@FWDekker FWDekker deleted the v2 branch November 17, 2023 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Code changes without behavior changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Symbolically merge v2 into main
1 participant