IPC144 VS IPC144

Reviewing two PRs from a classmate is one of the duties for this project. The PRs, as well as my reviews, are available here and here. Both of the PRs I reviewed were really well-written and detailed, with very few mistakes. It was interesting to observe how different people approached certain changes, such as adding a svg file.

For this assignment we had to contribute to a Seneca repository. I chose the IPC144 repo.

Issue: https://github.com/Seneca-ICTOER/IPC144/issues/64 I worked on the standardized front matter across all markdown pages. This open-source project is the C language course notes of my major program.

And the reason why the web-only artifacts is still appear on the PDF page is because the --excludeSelectors option is not implemented enough. I have to add .clean-btn to the --excludeSelectors, the purpose of this is not to include the "On the page" artifact in side the PDF page. This is the final source code I have implement for this improvement and my pull request

I think I was able to do a good job meeting my goals I gave myself in my planning phase of this release. I was able to finish the issues well on schedule while balancing my other courses like I hoped and I was able to properly audit and fix both issues #122 and #123 without needing too many changes after review. What I learned from those two issues is the importance to read and checkout other issues/pull requests, especially for smaller repos. As I was told in the review for both my issues, I learned that the project recently made changes with how we would format the frontmatter. In PR #142 we no longer use the slug for pages due to inconsistency with links and we also need to include a description to follow the standardized Frontmatter as updated in PR #143.

The first issue I was working on IPC144 Course Note is about improving the usability of the PDF file generated from the website. All contents are generated inside the PDF, however, we want to get this better since some of the pictures are not showing properly, and also the web-only artifacts are still on the PDF, which we do not want it when we use the "PDF" version. convert-to-pdf.sh file would be modified a bit to accomplish this.

Issue #113

PR

As for this pull request, the code review went much more smoothly, with me having to only make minor changes to ensure that it would not cause any errors when built.

2. #issue-107

My changes so far have been pretty minor, some language in a section on Unary Floating-Point operands was weird so I updated it to read a little better.

My latest open source endeavour has been to work on the updated Seneca College IPC144 course notes. IPC144 is a class I hold dear to my heart and have fond memories of, for introducing me to C and programming concepts in general. When I took the course back in 2017 I didn't have a laptop yet and would print out reams of these notes to study at work, so I'm intimately familiar with the notes and their endearingly dated aesthetic. I'm excited to be able to modernize them and help introduce them to the next crop of students who (studying from home I'm assuming) will hopefully have the privilege of viewing them on a computer as God intended, taking full advantage of the technical care myself and the other contributors have put into them.