History of my watch PR

I was working on Pagure, fixed few random issues. That boost me up so searched for more issues and thought why not go for adding a feature! I just selected one and asked Pingou that I want to work on this issue.

I started working on the issue. That was basically a feature for showing the users either you are watching or not the current repo and you can also change the current watch status for the repo. I added the markup, migrations, logics and pushed it. I was excited “Yea, Now I have done some coding!!”. But there was time for this thought. I started getting comments, those were not for “I have done a great coding!” rather for the fixes. Even I got comments for indentation fixes.

I started to work as per the comments and reached to a stage where I realized I must missing something that was nothing just my interest. I left working on this for few days and then restarted.

After 67 comments and lots of fixes, the PR got merged. In this bug fixing process I learned so many things like Alembic, SQLAlchemy, testing etc. Hopefully, I’ll see this feature on the next version of Pagure.

thanks Pingou.

Advertisements