Added ternary operator to allow sourcemaps on gatsby-cli develop

[BlackFenix2]

Description

I added a Ternary operator to allow the source-map devtool in development environments while using the 'gatsby-plugin-no-sourcemaps' plugin.

Why did i make this PR?

Sourcemaps are still totes useful for debugging when running gatsby develop

Could this be better?

yeah, if i could pass in a plugin option to configure sourcemap generation

Ex:

{
  resolve: `gatsby-plugin-no-sourcemaps`,
  options: {
    allowDev:true,
  }
}

why didn't you make this better 👀

... i don't know how to take plugin options as an argument 🤷‍♂. i'm welcome to suggestions.

Related Issues

kind of a half-way fix for #3817

[wardpeet]

Hey,

Thanks for your PR. It's better to do this condition inside gatsby-config and don't load the plugin on development.

You can checkout

gatsby/www/gatsby-config.js

Lines 40 to 54 in f594253

	if (process.env.AIRTABLE_API_KEY) {
	dynamicPlugins.push({
	resolve: `gatsby-source-airtable`,
	options: {
	apiKey: process.env.AIRTABLE_API_KEY,
	tables: [
	{
	baseId: `app0q5U0xkEwZaT9c`,
	tableName: `Community Events Submitted`,
	queryName: `CommunityEvents`,
	},
	],
	},
	})
	}

on how we do this in gatsbyjs.org.

const plugins = [];

if (process.env.NODE_ENV !== 'development') {
   plugins.push(`gatsby-plugin-no-sourcemaps`)
}

module.exports = {
  // gatsby config
  plugins,
};

Thanks again, and we look forward to seeing more PRs from you in the future! 💪💜