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.

Sign up for GitHub

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

Jump to bottom

Update scope methods to use kwargs #476

Merged

spohlenz merged 3 commits into main from scope-kwargs

Aug 28, 2024

Member

spohlenz commented Aug 28, 2024

Update scope methods to use kwargs where appropriate
Extract scopes block defaults from defaults option
Add Scopes::Definition#apply_options! method to avoid Demeter violation
Add grouped scopes example to sandbox users admin

spohlenz added 3 commits

August 28, 2024 10:55


          Add Scopes::Definition#apply_options! method to avoid Demeter violation

0c1c906


          Update scope methods to use kwargs where appropriate

a17e19a


          Add grouped scopes example to sandbox users admin

a9f5d2d

spohlenz merged commit 98d7272 into main

44 checks passed

spohlenz deleted the scope-kwargs branch

August 28, 2024 04:56

coveralls commented Aug 28, 2024

coverage: 91.474% (+0.01%) from 91.463%
when pulling a9f5d2d on scope-kwargs
into 6136ee6 on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet