Skip to content

feat: add support for extra-profiles-path option #103

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
Open
wants to merge 17 commits into
base: develop
Choose a base branch
from
Open

feat: add support for extra-profiles-path option #103

wants to merge 17 commits into from

Conversation

@kikkomep
Copy link
Member

@kikkomep kikkomep commented Jul 1, 2025

This PR addresses issue #96 by introducing support for an additional directory to load user-defined profiles through the new extra-profiles-path option.

Note that:

  • by default, the package’s built-in profiles are loaded first
  • when the extra-profiles-path option is set, profiles found in that location are loaded afterward
  • if a profile in the extra path shares the same identifier as a default profile, it will override the default one

kikkomep added 17 commits July 1, 2025 11:33
@kikkomep
feat(core): ✨ add support for overrides to the Profile model
ee07740
@kikkomep
feat(core): ⚡ automatically detect overrides during Profile initializ…
d92ead6 
…ation
@kikkomep
refactor(core): 🚚 rename internal method to load a single Profile ins…
00be4a6 
…tance
@kikkomep
feat(core): ✨ extend profiles loader to handle an extra loading path
d4a327d
@kikkomep
chore(core): 🎨 reformat code
ab0b7cd
@kikkomep
feat(services): ✨ extend services to support the extra profiles loadi…
8af6797 
…ng path
@kikkomep
feat(utils): ✨ add utility method to shorten a given path
c7e11f4
@kikkomep
feat(utils): ✨ add internal utility to shorten paths
24d8156
@kikkomep
feat(cli): ✨ add support for extra-ptofiles-path option in `profile…
ac9bd99 
…` command
@kikkomep
feat(cli): ✨ extend profile description to include name and overrides…
624c11a 
… info
@kikkomep
fix(core): 🐛 remove overridden profiles only if they are already loaded
be927d7
@kikkomep
feat(core): ✨ add support for the extra-profiles-paths in the `vali…
26ce28f 
…date` process
@kikkomep
feat(core): 🚸 improve profile loading when the `disable-profile-inher…
640083f 
…itance` is enabled
@kikkomep
feat(cli): ✨ add support for the extra-profiles-path option in the …
4926aa5 
…`validate` command
@kikkomep
test(services): 🧪 add unit tests for the services module
7df81cb
@kikkomep
test(cli): ✅ test extra-profiles-path option on CLI
7abf85c
@kikkomep
chore(test-conf): 🔧 replace old comment prefix in pytest config file
ed0778c
@kikkomep kikkomep mentioned this pull request Jul 1, 2025
Open
@ilveroluca
Copy link
Member

Can we add some documentation for this new feature before merging?

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kikkomep @ilveroluca