Rely on the `Linguist::Repository::MAX_TREE_SIZE` default value more (#6830)

* Rely on the Linguist::Repository::MAX_TREE_SIZE default value more

- Changing it won't be such a problem in future if there's just one place to change it. I hadn't realised it had propagated so much!

* Missed another hardcoded 100,000.
This commit is contained in:
Issy Long 2024-05-15 15:55:55 +01:00 коммит произвёл GitHub
Родитель d9b314022e
Коммит e2012cd678
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: B5690EEEBB952194
2 изменённых файлов: 7 добавлений и 5 удалений

Просмотреть файл

@ -10,7 +10,7 @@ require 'tempfile'
require 'zlib'
class GitLinguist
def initialize(path, commit_oid, incremental = true, tree_size = 100_000)
def initialize(path, commit_oid, incremental = true, tree_size = Linguist::Repository.MAX_TREE_SIZE)
@repo_path = path
@commit_oid = commit_oid
@incremental = incremental
@ -99,7 +99,7 @@ end
def git_linguist(args)
incremental = true
commit = nil
tree_size = 100_000
tree_size = Linguist::Repository::MAX_TREE_SIZE
parser = OptionParser.new do |opts|
opts.banner = <<~HELP
@ -112,7 +112,8 @@ def git_linguist(args)
opts.on("-f", "--force", "Force a full rescan") { incremental = false }
opts.on("-c", "--commit=COMMIT", "Commit to index") { |v| commit = v}
opts.on("-t", "--tree-size=NUMBER", Integer, "Maximum number of files scanned to detect languages (default: 100,000)" ) { |t| tree_size = t }
opts.on("-t", "--tree-size=NUMBER", Integer,
"Maximum number of files scanned to detect languages (default: #{Linguist::Repository::MAX_TREE_SIZE})" ) { |t| tree_size = t }
end
parser.parse!(args)

Просмотреть файл

@ -20,7 +20,7 @@ HELP
def github_linguist(args)
breakdown = false
json_output = false
tree_size = 100_000
tree_size = Linguist::Repository::MAX_TREE_SIZE
rev = 'HEAD'
path = Dir.pwd
@ -33,7 +33,8 @@ def github_linguist(args)
opts.on("-r", "--rev REV", String,
"Analyze specific git revision",
"defaults to HEAD, see gitrevisions(1) for alternatives") { |r| rev = r }
opts.on("-t", "--tree-size=NUMBER", Integer, "Maximum number of files scanned to detect languages (default: 100,000)") { |t| tree_size = t }
opts.on("-t", "--tree-size=NUMBER", Integer,
"Maximum number of files scanned to detect languages (default: #{Linguist::Repository::MAX_TREE_SIZE})") { |t| tree_size = t }
opts.on("-h", "--help", "Display a short usage summary, then exit") do
puts opts
exit