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

feat(stride): Stride outpost types and events #1914

Merged
merged 8 commits into from Oct 20, 2023

Conversation

Vvaradinov
Copy link
Contributor

Description


Closes ENG-2216

@Vvaradinov Vvaradinov requested a review from a team as a code owner October 19, 2023 11:34
@Vvaradinov Vvaradinov requested review from MalteHerrmann and GAtom22 and removed request for a team October 19, 2023 11:34
@linear
Copy link

linear bot commented Oct 19, 2023

Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

lgtm, minor comments

precompiles/outposts/stride/types.go Show resolved Hide resolved
precompiles/outposts/stride/types.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/types.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/events.go Dismissed Show dismissed Hide dismissed
precompiles/outposts/stride/events.go Dismissed Show dismissed Hide dismissed
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Merging #1914 (8fee2ee) into main (6201def) will decrease coverage by 0.51%.
The diff coverage is 11.66%.

❗ Current head 8fee2ee differs from pull request most recent head e632fe8. Consider uploading reports for the commit e632fe8 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1914      +/-   ##
==========================================
- Coverage   70.30%   69.80%   -0.51%     
==========================================
  Files         319      323       +4     
  Lines       23574    23764     +190     
==========================================
+ Hits        16574    16588      +14     
- Misses       6179     6354     +175     
- Partials      821      822       +1     
Files Coverage Δ
precompiles/common/types.go 44.68% <0.00%> (-17.09%) ⬇️
precompiles/outposts/stride/types.go 31.11% <31.11%> (ø)
precompiles/outposts/stride/events.go 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

Copy link
Contributor

@GAtom22 GAtom22 left a comment

Choose a reason for hiding this comment

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

LGTM! Great work @Vvaradinov !! Left a few comments

Please address the gosec warnings

precompiles/common/types.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/types.go Show resolved Hide resolved
Copy link
Contributor

@0xstepit 0xstepit left a comment

Choose a reason for hiding this comment

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

Great job @Vvaradinov! Left just minor comments

precompiles/common/types.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/events.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/types.go Outdated Show resolved Hide resolved
precompiles/outposts/stride/types.go Show resolved Hide resolved
@fedekunze fedekunze enabled auto-merge (squash) October 20, 2023 08:25
@fedekunze fedekunze merged commit f1ecc37 into main Oct 20, 2023
25 of 26 checks passed
@fedekunze fedekunze deleted the Vvaradinov/stride-outpost-types branch October 20, 2023 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants