Skip to content

environment: migrate more variables into repo_config_values - #14

Open
malon7782 wants to merge 2 commits into
masterfrom
more-variables
Open

environment: migrate more variables into repo_config_values#14
malon7782 wants to merge 2 commits into
masterfrom
more-variables

Conversation

@malon7782

Copy link
Copy Markdown
Owner

ca08a84a4b6a5fe9752745 Mon Sep 17 00:00:00 2001
From: Tian Yuchen cat@malon.dev
Date: Tue, 28 Jul 2026 09:34:34 +0800
Subject: [PATCH v2 0/2] environment: migrate more global variables into
repo_config_values, pt.2

Hi all,

This series moves:

  • (1/2) pack_size_limit_cfg
  • (2/2) assume_unchanged

into repo_config_values to continue the libification effort.

Change since v1:

  • drop the migration of minimum_abbrev and default_abbrev.

Related:

Thanks! yuchen

Tian Yuchen (2):
environment: migrate pack_size_limit_cfg into repo_config_values
environment: migrate assume_unchanged into repo_config_values

builtin/pack-objects.c | 2 +-
builtin/update-index.c | 3 ++-
environment.c | 8 ++++----
environment.h | 5 +++--
object-file.c | 5 +++--
read-cache.c | 9 ++++++---
6 files changed, 19 insertions(+), 13 deletions(-)

--
2.43.0

Move the global 'pack_size_limit_cfg' configuration into the
repository-specific 'repo_config_values' struct.

We do not introduce a getter for it because the readers are
limited and no hardcoded fallback values are needed.

Mentored-by: Christian Couder <christian.couder@gmail.com>
Mentored-by: Ayush Chandekar <ayu.chandekar@gmail.com>
Mentored-by: Olamide Caleb Bello <belkid98@gmail.com>
Signed-off-by: Tian Yuchen <cat@malon.dev>
Move the global 'assume_unchanged' configuration into the
repository-specific 'repo_config_values' struct.

We do not introduce a getter for it because the readers are
limited and no hardcoded fallback values are needed.

Mentored-by: Christian Couder <christian.couder@gmail.com>
Mentored-by: Ayush Chandekar <ayu.chandekar@gmail.com>
Mentored-by: Olamide Caleb Bello <belkid98@gmail.com>
Signed-off-by: Tian Yuchen <cat@malon.dev>
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