FreeCAD/FreeCAD

PartDesign: Hole thread pitch tests

Aberta

#21.195 aberto em 7 de mai. de 2025

Ver no GitHubGuia do colaborador
 (6 comentários) (0 reação) (1 responsável)C++ (5.550 forks)batch import
Help wantedMod: Part DesignType: Code Quality

Métricas do repositório

Stars
 (30.913 estrelas)
Métricas de merge de PR
 (Mesclagem média 24d 7h) (217 fundiu PRs em 30d)

Description

Is there an existing issue for this?

  • I have searched the existing issues

Problem description

The PR https://github.com/FreeCAD/FreeCAD/pull/21015 fixes an important issue but could be improved as discussed in the comments.

  • refactor to use std::ranges::min_element
  • prioritize same dimension over pitch
  • unit tests.

This issue is to track the small technical debt added by merging it before that's been fixed.

Full version info

N/A

Subproject(s) affected?

None

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Guia do colaborador

Direção de pesquisa
Implemente as melhorias: use std::ranges::min element, priorize a mesma dimensão sobre o passo e adicione testes de unidade. O ramo principal do desenvolvimento está em src/Mod/PartDesign/.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75