coders-school / desktop-business-app

0 stars 4 forks source link

README update: valgrind instructions #48

Closed mBialczak closed 7 months ago

mBialczak commented 7 months ago

Just readme update, following @fbtom suggestions to provide some instructions helping investigating (and prevent) memory leaks in our app & tests code (plase refer to issue #40)

ziobron commented 7 months ago

@fbtom, we do not have your data in the Coders School database.

  1. Please fill in this form https://forms.gle/rkzLsMUSBdnGQgdM9
  2. Edit this PR to run this automation once again (you can just add a space in the description or title).
  3. Wait one minute. If any label was added to this PR (like Standard, Premium, Podstawy C++, STL) then everything is fine :)
  4. If your solution passed, but nothing happens, modify your code (eg. add an empty line) to run the workflow again.

If you don't know what to do - ask on Discord and provide a link to this PR.

ziobron commented 7 months ago

Thanks @BaqWin for Code Review. 🏅 1 XP granted

ziobron commented 7 months ago

Your PR was merged! 🏅 2 XP granted. Thanks @mBialczak for making course materials better!