search

Karlheinzniebuhr / the-weather-scraper

A Lightweight Weather Scraper
108 stars 36 forks source link

Add Docker support #1

Closed yorch closed 4 years ago

kylepollina commented 4 years ago

Is there a reason to have docker support? Its just a simple python script - what is the necessary reason for this?

yorch commented 4 years ago

You may not have python installed, have the incorrect python version, or you may not want to install global dependencies in your python env using pip. Docker makes it more portable.

Karlheinzniebuhr commented 4 years ago

Hey, thanks for the PR but because this project has no binary dependencies, IMO adding docker adds more complexity than it takes away. I'll wait until it makes more sense to add.