For future reference, other than what was said here, I have a couple misc things to add. I have had my share of e30 m20 no starts.
1. If no crank, there is a wire that connects to the starter sometimes refered to as the "exciter" wire. Sometimes it gets corroded or loose. It has a 90* crimped connector on it.
2. There needs to be oil in the engine and the oil level sensor on the top of the oil pan needs to be plugged in. The car wont get an injector pulse.
3. Loose cap and rotor. Be careful with those bolts for the upper timing cover, they attach to the aluminum cylinder head. If they are loose, your going to missfire.
4. The crank position sensor and reference sensor connectors on the engine harness are very close and can be swapped accidentally.
5. Some years had an injector harness connector. This can get filled with corrosion and cause no injector pulse, but rare in my opinion.
6. Major vacuum leak.