Skip to content

Comments

feat: Control features with build arg#75

Open
AaronBenDaniel wants to merge 5 commits intofeature/#80-dockerfilefrom
feature/#74-unified-feature-branch
Open

feat: Control features with build arg#75
AaronBenDaniel wants to merge 5 commits intofeature/#80-dockerfilefrom
feature/#74-unified-feature-branch

Conversation

@AaronBenDaniel
Copy link
Collaborator

@AaronBenDaniel AaronBenDaniel commented Jun 24, 2025

Fixes #74.

Merges PDF and non-PDF branches and controls which features are enabled with a build argument.

By default, PDF downloads, the donator frontend, and unlimited bandwidth are disabled. If, while running docker build, --build-arg feature_flag=true--build-arg pdfs=true is added, the features will be enabled.

@AaronBenDaniel AaronBenDaniel self-assigned this Jun 24, 2025
@AaronBenDaniel AaronBenDaniel added the enhancement New feature or request label Jun 24, 2025
@AaronBenDaniel
Copy link
Collaborator Author

What about the --build-arg?

@TheOnlyWayUp TheOnlyWayUp changed the base branch from master to feature/#80-dockerfile October 28, 2025 13:19
@TheOnlyWayUp
Copy link
Owner

Rebasing to #82. 82 will be merged to master first, and this branch will be next.

@TheOnlyWayUp TheOnlyWayUp force-pushed the feature/#74-unified-feature-branch branch 2 times, most recently from 6822fb0 to ec16efe Compare October 28, 2025 13:59
@TheOnlyWayUp
Copy link
Owner

Rebased the 3 commits on this branch onto 82.
image

@TheOnlyWayUp TheOnlyWayUp force-pushed the feature/#74-unified-feature-branch branch from ec16efe to 2956399 Compare October 29, 2025 13:13
@AaronBenDaniel
Copy link
Collaborator Author

This branch is broken if #82 is merged. Fixes will need to be made before this can get merged into master.

@AaronBenDaniel
Copy link
Collaborator Author

I believe this branch is ready to merge. @TheOnlyWayUp can you review it?

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unified feature branch

2 participants