Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

✨ isEnabled checks (config.)enabled; developer bypass .env #85

Merged
merged 2 commits into from
May 19, 2021

Conversation

JeromeFitz
Copy link
Owner

This allows for developers to override a repo with a .env process flag that
the team can agree to put in changelog.config.js

.env => GIT_CZ = false

Then in changelog.config.js can pass enabled: process.env.GIT_CZ || true

🚨️ fix some lint issues and ordering with yarn gitmoji and *.js filenames

This allows for developers to override a repo with a `.env` process flag that
the team can agree to put in `changelog.config.js`

.env => GIT_CZ = false

Then in `changelog.config.js` can pass `enabled: process.env.GIT_CZ || true`
@@ -24,6 +24,7 @@ const setState = (cliOptions) => {
// @todo(uhh) this cannot be right, haha
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea should look at this 😆

full disclosure this shoudl be an extend ont the way this is done
@JeromeFitz JeromeFitz added the 🥳️ LGTM Automerge: Let's Get This Merged label May 19, 2021
@kodiakhq kodiakhq bot merged commit 7736957 into main May 19, 2021
@JeromeFitz
Copy link
Owner Author

🎉 This PR is included in version 8.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🥳️ LGTM Automerge: Let's Get This Merged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant