Skip to content

Command ignore#3

Merged
adamgracikowski merged 5 commits into
developfrom
feature/command-ignore
Jul 27, 2025
Merged

Command ignore#3
adamgracikowski merged 5 commits into
developfrom
feature/command-ignore

Conversation

@adamgracikowski
Copy link
Copy Markdown
Contributor

Co w PR?

Zmiany w cli:

  • Polecenie ignore wraz ze wszystkimi sub-poleceniami (zgodnie z plikiem ignore.md specki).
  • Trait OpenInEditor, w którym wyodrębniłem logikę otwierania edytora (bo powtarza się przy otwieraniu konfiga, pliku do ignorowania i jeszcze niedługo przy commit może się przydać)

Zmiany w engine:

  • Trait IgnoreOperations ze zdefiniowanymi operacjami na plikach ignorujących.
  • IgnoreResult, który mówi, czy plik jest ignorowany, czy nie i z jakiego pliku pochodzi match.
  • IgnoreService, który implementuje IgnoreOperations i jest używany w poleceniach w cli.

Uwaga:
Ustawiam narazie na draft, bo testy jednostkowe są jeszcze bardziej ubogie niż poprzednio xD.
Ale możesz potestować sobie manualnie. Biblioteczka użyta to: globset, więc warto sobie popatrzeć jakie patterny są valid.

Przykładowo ja sobie testowałem na takim:

# Ignoruj wszystkie pliki tymczasowe
*.tmp

# Ignoruj katalogi build na dowolnej głębokości
**/build/

# Ignoruj plik config w głównym folderze
/config.json

# Ignoruj wszystkie pliki .log w katalogu logs lub jego podkatalogach
logs/**/*.log

# Ignoruj dokładnie plik `.DS_Store`
.DS_Store

# Ignoruj pliki foo lub bar
{foo,bar}

# Ignoruj pliki kończące się na `.bak` lub `.old`
*.{bak,old}

# Ignoruj folder cache i wszystko co w nim
cache/**
test.rs

*.sh

W konsolce to wygląda mniej więcej tak (całkiem przyjemnie mam wrażenie):

pr

@adamgracikowski
Copy link
Copy Markdown
Contributor Author

Dodałem trochę unit testów, fajnie gdybyś dzisiaj obejrzał, bo chciałbym wprowadzić ewentualne poprawki i potem mergować 😎

@adamgracikowski adamgracikowski marked this pull request as ready for review July 25, 2025 13:59
@adamgracikowski adamgracikowski merged commit 0c9f0a1 into develop Jul 27, 2025
1 check passed
@adamgracikowski adamgracikowski deleted the feature/command-ignore branch September 14, 2025 07:17
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.

2 participants