wojtryb / Shortcut-Composer

Krita plugin for creating complex keyboard shortcuts
GNU General Public License v3.0
261 stars 9 forks source link

Tools to control blending of Color Sampler Tool #22

Closed SirPigeonz closed 1 year ago

SirPigeonz commented 1 year ago
SirPigeonz commented 1 year ago

Poprawione. Użyłem ammend last commit nie wiem, czy to poprawna metoda xD

wojtryb commented 1 year ago

Dzięki, jeszcze nie mam pomysłu czy będę wydawać 1.1.1 czy 1.2.0 (ani co powinno się w niej znaleźć, więc póki co jeszcze nie merguje)

Ammend jest spoko kiedy odkryjesz jakąś literówkę albo buga po zrobieniu git push. Wtedy nie musisz robić małych commitów w stylu "Fixing typo". Po CR lepiej jest zrobić zwykłego commita na tym samym branchu i wtedy zwykłe git push dorzuci go do pull requesta, a robiący CR widzi zmiany jakie się zadziały od CR.

Generalnie commitów lepiej jest chyba mieć za dużo niż za mało, bo przy mergowaniu zawsze mogę zrobić squash i spłaszczyć je do jednego, albo można zrobić rebase i zacząć je usuwać, zmieniać nazwy, kolejność i łączyć jeżeli zrobi Ci się w nich syf.

Force-push przydaje się gdy chcesz zrobić rebase - uładniając historię commitów, albo przestawiając je na innego commita początkowego.

Jakiego programu używasz żeby edytować ten kod i obsługiwać gita?

SirPigeonz commented 1 year ago

Spoko, niech czeka na poprawny release. :)

Dzięki za tipy!

Jakiego programu używasz żeby edytować ten kod i obsługiwać gita?

Do edycji używam VSCode a do obsługi Gita używam GitHubowego graficznego programu.

wojtryb commented 1 year ago

Najlepiej :)

Spoko jakbyś pod F1 > Open Workspace Settings (JSON) wrzucił sobie:

{
    "editor.rulers": [
        72,
        79
    ],
    "python.formatting.provider": "autopep8",
    "python.linting.flake8Enabled": true,
    "editor.formatOnSave": true,
    "python.analysis.typeCheckingMode": "basic",
    "python.analysis.diagnosticSeverityOverrides": {
        "reportMissingImports": "none"
    }
}

To wtedy będziesz miał ze mną zgodne formatowanie. Jak sobie zrobisz ctrl+s z tymi ustawieniami to Ci to ładnie sformatuje. Przejrzyj jeszcze te wszystkie pliki które dodałeś, bo widzę teraz, że w paru miejscach jeszcze lecą błędy z lintera.

Do gita bardzo polecam rozszerzenie git graph. W pasku na dole pojawia Ci się przycisk otwierający okno z drzewem gitowym - klikając prawym na commity możesz się na nie przełaczać, tworzyć branche, a klikając prawym podglądać jakie zmiany były w nim wprowadzone. A samo tworzenie commitów i pushowanie ich też da się wygodnie robić z poziomy VsCode w odpowiedniej zakładce po lewej stronie (oczywiście przy założeniu, że otwarłeś całe repozytorium jako projekt).

SirPigeonz commented 1 year ago

O dzięki :D Tak zrobię! Usiądę do tego dzisiaj pewnie, przez ostatnie 2 dni ogarniałem rzeczy by wrócić do kodzenia Krity. Rozruszałeś moje programistyczne stawy dzięki :D

wojtryb commented 1 year ago

O dzięki :D Tak zrobię! Usiądę do tego dzisiaj pewnie, przez ostatnie 2 dni ogarniałem rzeczy by wrócić do kodzenia Krity. Rozruszałeś moje programistyczne stawy dzięki :D

Widziałem na czacie, że Agata pomagała Ci postawić środowisko i że poszedł już jakiś PR. Fajnie :)

Chcesz, żeby Cię dodać Cię do tego repozytorium jako kontrybutora, czy wolisz dalej pracować ze swojego forka?

SirPigeonz commented 1 year ago

Widziałem na czacie, że Agata pomagała Ci postawić środowisko i że poszedł już jakiś PR. Fajnie :)

Tak :D zabawne jest to, że to już drugi raz mi pomaga. Ostatnio robiliśmy to samo 9 miesięcy temu i wtedy też poszedł PR który był dodany do ostatniej stabilnej wersji. Ledwo mnie pamiętała :D (9 miechów temu myślałem że to koleś xD) Teraz planuje regularnie coś dłubać by znowu nie zardzewieć, co prawda poszło szybciej niż ostatnio bo dużo pamiętałem, ale to jednak irytujące wdrażać się na nowo xD

Będę teraz pomagał z jej projektem by usprawnić Assistant'y. Mam też kilka osobistych projektów, które powoli dłubie: usprawnienia Sketch Engine (<3 kocham ten silnik), Node Brush Engine, chce też zerknąć jak działa integracja z Pythonem jakbyśmy coś potrzebowali tutaj :P

Niestety dalej nie udało mi się ładnie skonfigurować VSCode dla Krity, więc używam QtCreator'a jest całkiem przyjemny, ale VSCode to moja nowa miłość, potem to ogarnę. Zabawne jest to, że twój plugin to pierwszy kod na którym go wypróbowałem. xD

W ogóle dowiedziałem się czemu Document i Node mają osobno metody na tworzenie i dodawanie nodeów, a C++ kod ma klasę KisNodeManager która potrafi obie rzeczy na raz. Klasa Document jest bardziej do pisania importerów, myśleli, że Python będzie głównie używany do integracji z pipelinami dlatego tak to jest zrobione.

To Ci nagadałem xD

Chcesz, żeby Cię dodać Cię do tego repozytorium jako kontrybutora, czy wolisz dalej pracować ze swojego forka?

Będę zaszczycony! Jak wygląda wtedy workflow? Robię branch'a i mogę tak samo wystawić PR do mastera, czy jakoś inaczej? Dodatkowo odpadnie fetching upstream heh xD