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

Pass config file as argument to build command #156

Merged
merged 3 commits into from
Apr 5, 2025

Conversation

udit-001
Copy link
Contributor

@udit-001 udit-001 commented Mar 19, 2025

Possible fix for #155

@oliverandrich
Copy link
Member

Can you please update the test code too? Then I happy to merge this asap.

@udit-001
Copy link
Contributor Author

udit-001 commented Apr 1, 2025

Sure, I'll take a look. I initially thought you were working on a different implementation for this.

I was also wondering if the config flag needs to be passed to the watch command as well.

Let me know if that's necessary, and I'll go ahead and modify it accordingly.

@udit-001 udit-001 force-pushed the fix/custom-config branch from 92da3ec to 4542580 Compare April 1, 2025 20:08
@oliverandrich oliverandrich merged commit 07c5523 into django-commons:main Apr 5, 2025
4 checks passed
@oliverandrich
Copy link
Member

Thanks for the contribution.

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