search

mbadry1 / DeepLearning.ai-Summary

This repository contains my personal notes and summaries on DeepLearning.ai specialization courses. I've enjoyed every little bit of the course hope you enjoy my notes too.
MIT License
5.17k stars 2.44k forks source link

Remove all trailing whitespaces #158

Closed iiey closed 5 years ago

iiey commented 5 years ago

Trailing whitespaces add so much noise on diff files. So I make an update for removing all of them. This change has no effect on current markdown presentation. Hope you could accept it :)

VladKha commented 5 years ago

I agree, that there are probably some extra whitespaces, but in some places they are put on purpose for improving the formatting And your commit will destroy some formatting like here:

screenshot of example before your commit: screen shot 2019-01-07 at 16 31 48

screenshot of example if we merge your commit: screen shot 2019-01-07 at 16 31 41

iiey commented 5 years ago

@VladKha thanks for the fast respond and check. In this case, I could use <br\> for linebreak, so all images with show up correctly as before. I find, double spaces trick for linebreak is not a clean solution.

Did you find other formatting problems due to this change?

VladKha commented 5 years ago

@iiey I am not sure that <br\> is a cleaner solution than triple spaces But let someone else cast his/her opinion

It was the first one that got into my sight, maybe there are more - not sure