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

perf: efficient describe calls #141

Merged
merged 3 commits into from
Mar 8, 2024
Merged

perf: efficient describe calls #141

merged 3 commits into from
Mar 8, 2024

Conversation

dschach
Copy link
Owner

@dschach dschach commented Mar 8, 2024

feat: enable case-insensitive uses

What does this PR do?

(check all applicable)

  • ❗ BREAKING CHANGE
  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Describe your changes

Avoid using Global Describe
Update all methods to use lowercase keys so inputs are no longer case-sensitive

Checklist before requesting a review

  • πŸ• I have performed a self-review of my code
  • πŸ“— The code has been linted and formatted using the repo Prettier configuration
  • βœ… If it includes Apex, Aura, LWC, or Flows, I have added thorough tests
  • πŸ“ I have used descriptive commit messages

Added to documentation?

  • πŸ“œ README.md
  • πŸ™… no documentation needed

By submitting this PR, you agree to follow the Code of Conduct

  • I agree to follow this project's Code of Conduct

Copy link

codecov bot commented Mar 8, 2024

Codecov Report

Attention: Patch coverage is 99.21260% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 99.16%. Comparing base (90ea466) to head (2a72435).
Report is 119 commits behind head on main.

Files Patch % Lines
force-app/main/default/classes/RecordTypes.cls 99.21% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #141      +/-   ##
==========================================
+ Coverage   98.48%   99.16%   +0.68%     
==========================================
  Files           1        2       +1     
  Lines         132      240     +108     
==========================================
+ Hits          130      238     +108     
  Misses          2        2              
Flag Coverage Ξ”
Apex 99.16% <99.21%> (+0.68%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@dschach dschach merged commit 3eadfcf into main Mar 8, 2024
18 checks passed
@dschach dschach deleted the feat/efficient-describe branch March 8, 2024 23:48
This pull request was closed.
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.

1 participant