Skip to content

feat: adicionar pacote setter e integrar com request#41

Merged
gfecchio merged 1 commit into
mainfrom
zzzep/issue16
Feb 9, 2026
Merged

feat: adicionar pacote setter e integrar com request#41
gfecchio merged 1 commit into
mainfrom
zzzep/issue16

Conversation

@gfecchio
Copy link
Copy Markdown
Owner

@gfecchio gfecchio commented Feb 9, 2026

📝 Descrição

Adiciona um novo pacote setter com funções utilitárias para definir valores apenas quando não estão vazios, e integra com o pacote request.

✨ Funcionalidades

Novo Pacote Setter

  • SetIfNotEmpty[T] - Função genérica para qualquer tipo comparável
  • SetIfNotEmptyString - Específica para strings
  • SetIfNotEmptySlice - Específica para slices
  • SetIfNotEmptyMap - Específica para maps

Integração com Request

  • SetUrl agora usa SetIfNotEmptyString para preservar URL existente quando nova é vazia
  • SetHeader otimizado para não processar maps vazios

🔄 Mudanças

  • ✅ Criado pacote setter com funções utilitárias
  • ✅ Adicionados testes completos para todas as funções
  • ✅ Adicionados exemplos de uso documentados
  • ✅ Integrado setter no pacote request
  • ✅ Adicionados testes para verificar comportamento
  • ✅ Atualizado README.md com nova seção SETTER

📋 Arquivos Modificados

Novos Arquivos

  • setter/setter.go
  • setter/setter_test.go
  • setter/example_test.go

Arquivos Modificados

  • request/http.go - Integração com setter
  • request/example_test.go - Novos testes
  • README.md - Documentação atualizada

✅ Verificações

  • Todos os testes passando
  • Sem erros de lint
  • Exemplos documentados
  • Código seguindo padrões do projeto

- Criado novo pacote setter com funções SetIfNotEmpty
- Implementadas funções para string, slice, map e tipos genéricos
- Integrado setter no pacote request para SetUrl
- Adicionados testes e exemplos de uso
- Atualizado README.md com nova seção SETTER
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2026

Codecov Report

❌ Patch coverage is 84.21053% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
request/http.go 25.00% 2 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
setter/setter.go 100.00% <100.00%> (ø)
request/http.go 73.23% <25.00%> (-1.05%) ⬇️

... and 1 file with indirect coverage changes

@gfecchio gfecchio merged commit afb4714 into main Feb 9, 2026
1 check passed
@gfecchio gfecchio deleted the zzzep/issue16 branch February 9, 2026 15:06
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