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

fix: add .rpc for consistency in engine settings #4158

Merged
merged 1 commit into from Oct 24, 2023

Conversation

kylezs
Copy link
Contributor

@kylezs kylezs commented Oct 24, 2023

Checklist

Please conduct a thorough self-review before opening the PR.

  • I am confident that the code works.
  • I have updated documentation where appropriate.

Summary

Fixes an inconsistency in the cli config items... need to fix the release docs too to reflect the actual settings.

@kylezs kylezs changed the title fix: add .rpc for consistency fix: add .rpc for consistency in engine settings Oct 24, 2023
@kylezs kylezs enabled auto-merge (squash) October 24, 2023 12:11
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #4158 (12ac61b) into main (f9bf5e5) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #4158   +/-   ##
=====================================
- Coverage     72%     72%   -0%     
=====================================
  Files        378     378           
  Lines      60921   60921           
  Branches   60921   60921           
=====================================
- Hits       43749   43744    -5     
- Misses     14901   14905    +4     
- Partials    2271    2272    +1     
Files Coverage Δ
engine/src/settings.rs 86% <ø> (ø)

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kylezs kylezs merged commit 9cb4669 into main Oct 24, 2023
42 checks passed
@kylezs kylezs deleted the fix/basic-auth-settings branch October 24, 2023 12:40
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

2 participants