Skip to content

Add placeholder function so executing scripts with inbuilt unit tests don't error#65

Open
t-nil wants to merge 2 commits intojorgebucaran:mainfrom
t-nil:patch-1
Open

Add placeholder function so executing scripts with inbuilt unit tests don't error#65
t-nil wants to merge 2 commits intojorgebucaran:mainfrom
t-nil:patch-1

Conversation

@t-nil
Copy link
Copy Markdown

@t-nil t-nil commented Jan 16, 2024

It seems also, that your current form of creating these functions (@test, @echo) in fishtape and string collecting them into the newly spawned shell wouldn't work anymore, because you erase those functions globally. Maybe putting them in a special file.

Or actually, just making them available to the user, ergo moving them to functions/@{echo,test}.fish would also be totally acceptable from my perspective. They interfere with global function space anyway. Then an env could be added to check if the script is called by fishtape.

What do you think?

@jorgebucaran
Copy link
Copy Markdown
Owner

Could you clarify what you mean when you say the functions would no longer work in the newly spawned shell? As far as I am concerned, Fishtape has been functioning correctly for at least the past three years. Has something changed or broken?

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.

2 participants