jmtellez/cli-mate

Node CLI app that gives you the weather forecast for a given city.

 

CLI-mate 🌅

Command Line application that gives you the weather info for a given city/place/address/airport code ☀️

Install

npm install -g @jmt3559/cli-mate

On your terminal export these two environment variables:

Zsh

echo 'export MAPBOX=API_KEY' >>  ~/.zprofile

echo 'export WEATHERSTACK=API_KEY' >> ~/.zprofile

Bash

echo 'export MAPBOX=API_KEY' >>  ~/.bash_profile

echo 'export WEATHERSTACK=API_KEY' >> ~/.bash_profile

Get API Keys

Weatherstack - 1k free requests per month

Mapbox - 100K free requests

cli-mate's Issues

Options order

Work on the order on how the location and units are passed. Order should not matter:

$ cli-mate Dallas --u=m
$ cli-mate --u=m Dallas

both should return the same output:

✔ Dallas, Texas, United States
Partly cloudy. It is currently 23°C, it feels like 24°C.

Error when API keys are not set

Following Error is thrown if API keys are not set in the environment variables

This can be solved by checking if process.env.WEATHERSTACK and process.env.MAPBOX exists

Update Notifier

Add update-notifier to package to check for new available versions.

For reference: https://www.npmjs.com/package/update-notifier

Adding Semantics for releases

Pre-requisites:

• Yes, I looked through both open and closed issues looking for what I needed

Feature Description

Would like to add workflow and integrate semantics-release to take care of versioning of package

Additional Details

Refactoring entry file

Moving some code to utils directory for clean up.

CONTRIBUTING.md

Work on CONTRIBUTING.md file to guide contributors in following rules and standards for this project.

Units Flag

Weather data is returned in Fahrenheit.
Add flag to pass desired units (check WeatherStack API for units)

Add Issue, feature request, and author files

Create folder and add templates for the following:

• Issues
• Feature request
• Authors

Geolocation

When no command is passed, cli-mate should return weather data from where the user is located.

Add emojis to response

Pre-requisites:

• Yes, I looked through both open and closed issues looking for what I needed

Feature Description

Adding emojis to illustrate the weather description

Additional Details

node-emoji as reference

Cannot read property 'length' of undefined

➜  ~ cli-mate mcallen
⠸ Getting geocodes/usr/local/lib/node_modules/@jmt3559/cli-mate/utils/geocode.js:14
      } else if (body.features.length === 0) {
                               ^

TypeError: Cannot read property 'length' of undefined

Possible REST API parse error - I'll try to fix this, but put it out here so others can see it.
Edit: no, it was because I don't have a mapbox api key. We should specify that both api keys are required for cli-mate to function.

Display additional information from weatherstack

Leverage weather data from the rest call being done to Weatherstack
Think of additional information we can display in the correct format that attracts the user.

setTimeout() wraps all network calls?

Why do all of the network calls happen after a 1second setTimeout()?

If it's a question of the state of the request connection, according to the documentation on https://github.com/postmanlabs/postman-request#requestoptions-callback - request's options object takes a timeout: value which should cover this.

Otherwise, I'd suggest perhaps working with returning a Promise or an Async/Await style for http requests?

Update the README

Pre-requisites:

• Yes, I looked through both open and closed issues looking for what I needed

Feature Description

Please update the readme docs for us windows users. IDK HOW TO INSTALL!!!!!

Additional Details

Please.