Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

refactor(config): remove polaris.toml to simplify infra for brick #1130

Merged
merged 8 commits into from
Sep 20, 2023

Conversation

itsdevbear
Copy link
Contributor

No description provided.

@itsdevbear itsdevbear marked this pull request as ready for review September 20, 2023 17:58
@itsdevbear
Copy link
Contributor Author

@calbera @BrickBera should be ready

@BrickBera you'll need to modify your infra

@itsdevbear itsdevbear requested a review from ocnc September 20, 2023 18:18
@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Merging #1130 (45ac5fa) into main (a6b99f6) will decrease coverage by 0.01%.
The diff coverage is 22.22%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1130      +/-   ##
==========================================
- Coverage   54.79%   54.79%   -0.01%     
==========================================
  Files         101      101              
  Lines        5283     5265      -18     
==========================================
- Hits         2895     2885      -10     
+ Misses       2217     2209       -8     
  Partials      171      171              
Files Changed Coverage Δ
e2e/testapp/app.go 1.66% <0.00%> (+0.07%) ⬆️
cosmos/x/evm/keeper/keeper.go 49.27% <100.00%> (-6.43%) ⬇️

Copy link
Contributor

@calbera calbera left a comment

Choose a reason for hiding this comment

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

lgtm, will update bera to this as soon as in

@itsdevbear
Copy link
Contributor Author

just wrangling tests

@itsdevbear itsdevbear changed the title refactor(config): remove polaris.toml to simplify infra for brick refactor(config): remove polaris.toml to simplify infra for brick Sep 20, 2023
@itsdevbear itsdevbear added the merge me daddy Trigger Beradozer to bulldoze the PR label Sep 20, 2023
@itsdevbear itsdevbear removed the merge me daddy Trigger Beradozer to bulldoze the PR label Sep 20, 2023
auto-merge was automatically disabled September 20, 2023 18:54

Merge queue setting changed

@itsdevbear itsdevbear added the merge me daddy Trigger Beradozer to bulldoze the PR label Sep 20, 2023
@itsdevbear
Copy link
Contributor Author

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Sep 20, 2023

refresh

✅ Pull request refreshed

@itsdevbear itsdevbear merged commit ef65301 into main Sep 20, 2023
14 of 16 checks passed
@itsdevbear itsdevbear deleted the config-toml branch September 20, 2023 18:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
merge me daddy Trigger Beradozer to bulldoze the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants