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

use resolved paths for config and scripts #5

Merged
merged 1 commit into from
Jan 12, 2024
Merged

Conversation

sarnold
Copy link
Owner

@sarnold sarnold commented Jan 10, 2024

  • return resolved path obj from load_config, use it in get_userscripts
  • update tests for full script path in get_userscripts list

@sarnold sarnold force-pushed the full-cfg-path branch 2 times, most recently from 693ecd5 to e4442f5 Compare January 10, 2024 04:55
* return resolved path obj from load_config, use it in get_userscripts
* update tests for full script path in get_userscripts list

Signed-off-by: Stephen L Arnold <[email protected]>
Copy link

Package Line Rate Branch Rate Complexity Health
procman 90% 76% 0
Summary 90% (82 / 91) 76% (19 / 25) 0

Copy link

Hello @sarnold! Thanks for opening this PR. We found the following information based on analysis of the coverage report:

Base Branch Rate coverage is 76%
Merging e9616c0 into master will not change coverage

Nice work, @sarnold. Cheers! 🍻

@sarnold sarnold merged commit 956e1cc into master Jan 12, 2024
45 checks passed
@sarnold sarnold deleted the full-cfg-path branch March 16, 2024 04:43
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