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

allow multiple base packages, comma-separated #1154

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

ttulka
Copy link
Contributor

@ttulka ttulka commented Jan 20, 2025

For our services, we want a central SpringWolf configuration coming from a shared lib.
This is not possible with a single base package as the services have different base packages with no common root.

This PR tries to improve that.

Thanks!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to Springwolf. Thanks a lot for creating your first pull request. Please check out our contributors guide and feel free to join us on discord.

Copy link

netlify bot commented Jan 20, 2025

Deploy Preview for springwolf-ui canceled.

Name Link
🔨 Latest commit 872158d
🔍 Latest deploy log https://app.netlify.com/sites/springwolf-ui/deploys/678e7966ce24a00008217b38

@timonback timonback merged commit 12c504c into springwolf:master Jan 24, 2025
5 checks passed
@timonback
Copy link
Member

Thank you @ttulka for this addition!

This will be included in the next release.
Can you open a PR in the documentation about this enhancement? https://github.com/springwolf/springwolf.github.io/edit/master/docs/configuration/configuration.mdx

@ttulka
Copy link
Contributor Author

ttulka commented Jan 28, 2025

Hi @timonback, thanks for merging! This will help us a lot :)
I put the information to the doc as well
Cheers

@timonback
Copy link
Member

@all-contributors add @ttulka for code

(I am testing the all-contributors bot, nothing to do from your side)

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.

2 participants