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

chore: Extract (connor, crdt, core) errors to errors.go #968

Merged
merged 5 commits into from
Dec 15, 2022

Conversation

AndrewSisley
Copy link
Contributor

Relevant issue(s)

Part of #257

Description

Extracts inline (connor, crdt, core) errors to errors.go. A few of the connor errors were incorrect and have been changed.

@AndrewSisley AndrewSisley added area/errors Related to the internal management or design of our error handling system action/no-benchmark Skips the action that runs the benchmark. labels Dec 15, 2022
@AndrewSisley AndrewSisley added this to the DefraDB v0.4 milestone Dec 15, 2022
@AndrewSisley AndrewSisley requested a review from a team December 15, 2022 01:06
@AndrewSisley AndrewSisley self-assigned this Dec 15, 2022
@codecov
Copy link

codecov bot commented Dec 15, 2022

Codecov Report

Merging #968 (0f69d66) into develop (5eb6dbc) will decrease coverage by 0.02%.
The diff coverage is 6.45%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #968      +/-   ##
===========================================
- Coverage    59.86%   59.83%   -0.03%     
===========================================
  Files          157      160       +3     
  Lines        17716    17724       +8     
===========================================
  Hits         10605    10605              
- Misses        6157     6165       +8     
  Partials       954      954              
Impacted Files Coverage Δ
connor/and.go 58.33% <0.00%> (ø)
connor/connor.go 92.30% <0.00%> (ø)
connor/errors.go 0.00% <0.00%> (ø)
connor/ge.go 58.33% <0.00%> (ø)
connor/gt.go 60.00% <0.00%> (ø)
connor/in.go 58.33% <0.00%> (ø)
connor/le.go 58.33% <0.00%> (ø)
connor/lt.go 61.11% <0.00%> (ø)
connor/or.go 58.33% <0.00%> (ø)
core/crdt/base.go 73.07% <0.00%> (ø)
... and 5 more

@AndrewSisley AndrewSisley changed the title chore: Extract inline (connor, crdt, core) errors to errors.go chore: Extract (connor, crdt, core) errors to errors.go Dec 15, 2022
@AndrewSisley AndrewSisley changed the title chore: Extract (connor, crdt, core) errors to errors.go chore: Extract (connor, crdt, core) errors to errors.go Dec 15, 2022
Copy link
Member

@shahzadlone shahzadlone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much cleaner, I like the changes.

@AndrewSisley AndrewSisley merged commit f835e27 into develop Dec 15, 2022
@AndrewSisley AndrewSisley deleted the sisley/chore/I257-errors-go-2 branch December 15, 2022 16:01
shahzadlone pushed a commit to shahzadlone/defradb that referenced this pull request Feb 23, 2024
…k#968)

* Extract connor errors to errors.go

* Extract crdt errors to errors.go

* Move crdt error to crdt package

* Remove ineffective WithStack call

These errors are already defra errors and already have a stacktrace attached

* Extract core errors to errors.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/no-benchmark Skips the action that runs the benchmark. area/errors Related to the internal management or design of our error handling system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants