Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the ON::INIT script idea #38

Merged
merged 1 commit into from
Oct 13, 2019
Merged

Conversation

jacobdeichert
Copy link
Owner

Describe the solution

My original goal was to allow each command environment to be able to share some common config and helpers. The on::init script would prepare the environment, however, I ran into some weird issues with zsh not being able to export functions and bash set -a causing subshells invoked by commands (like git stash) to completely fail.

I'll need to put some more thought into allowing the sharing of common env variables and helpers.

Types of changes

  • Bug fix
  • New feature
  • Breaking change

My original goal was to allow each command environment to be able to share some common config/helpers. The on::init script would prepare the environment, however, I ran into some weird issues with zsh not being able to export functions and bash set -a causing subshells invoked by commands (like git stash) to completely fail.

I'll need to put some more thought into allowing the easy sharing of common env variables and helpers.
@jacobdeichert jacobdeichert changed the title Remove the bad on::init script idea Remove the bad ON::INIT script idea Oct 13, 2019
@jacobdeichert jacobdeichert changed the title Remove the bad ON::INIT script idea Remove the ON::INIT script idea Oct 13, 2019
@jacobdeichert jacobdeichert merged commit 2a627df into master Oct 13, 2019
@jacobdeichert jacobdeichert deleted the remove-init-script branch April 4, 2020 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant