Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update PIT dla zatrudnionych Art. 14. #49

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

aswsar
Copy link

@aswsar aswsar commented May 14, 2023

Po co drugi raz podawać stawkę podatku?
Wydaje się być rozsądnym doprecyzowanie.

@piotrholda
Copy link

To nie powinien być PR tylko Issue.

@aswsar
Copy link
Author

aswsar commented May 14, 2023

To nie powinien być PR tylko Issue.

Założyłem Issue i podpiąłem PR. Nie znalazłem Readme jaka jest koncepcja proponuje zmiany do ustawy na github. Osobiście tak wydaje mi się sensowniej przy konkretnych propozycjach zmiany - od razu widać o co mi chodzi.

Teraz jest ok, czy powinienem różnice wkleić do Issue i wywalić PR?

@czlowiek488
Copy link

czlowiek488 commented May 15, 2023

@aswsar Ja korzystam w projektach open source z takiej konwencji.

  1. Otwieram issue i czekam na opinie
  2. Jak społeczność jest chętna to otwieram PR

Tutaj to akurat nie ma znaczenia, ale w przypadku większych zmian to nieszczególnie fajnie jak zostaną one odrzucone. Po prostu marnuje się czas na pracę z której nikt nie skorzysta.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants