fac-17 / PFAK-week5

Country - News App
https://blooming-ridge-78902.herokuapp.com/
0 stars 2 forks source link

Declaring variables and not using them #28

Closed crianonim closed 5 years ago

crianonim commented 5 years ago

For example: handler.js: line 5

const env = require("dotenv").config();

Since you never use env you do not need to create a global variable

dubhcait commented 5 years ago

ya can just write

require("dotenv").config();

Plus further down line 46 const newsKey = process.env.DBAPIKEYNEWS; DB is usually used for databases, not APIs, it's just naming convention.