Skip to content

fix(direnv): set ERR trap after definition & before use#86571

Closed
bukzor wants to merge 1 commit intomasterfrom
bukzor/direnv-trap-better
Closed

fix(direnv): set ERR trap after definition & before use#86571
bukzor wants to merge 1 commit intomasterfrom
bukzor/direnv-trap-better

Conversation

@bukzor
Copy link
Copy Markdown
Contributor

@bukzor bukzor commented Mar 7, 2025

Also, fix the return code, which was making bash4 quite sad.

Also, fix the return code, which was making bash4 quite sad.
@bukzor bukzor requested a review from a team as a code owner March 7, 2025 00:05
@bukzor bukzor requested a review from joshuarli March 7, 2025 00:05
@bukzor bukzor requested a review from a team as a code owner March 7, 2025 00:05
@bukzor bukzor enabled auto-merge (squash) March 7, 2025 00:06
@asottile-sentry
Copy link
Copy Markdown
Contributor

is this fixing something? it's not clear why this needs to change

@getsantry
Copy link
Copy Markdown
Contributor

getsantry bot commented Jun 6, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added Stale and removed Stale labels Jun 6, 2025
@getsantry
Copy link
Copy Markdown
Contributor

getsantry bot commented Jun 29, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added Stale and removed Stale labels Jun 29, 2025
@getsantry
Copy link
Copy Markdown
Contributor

getsantry bot commented Jul 22, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added Stale and removed Stale labels Jul 22, 2025
@getsantry
Copy link
Copy Markdown
Contributor

getsantry bot commented Aug 14, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added Stale and removed Stale labels Aug 14, 2025
@getsantry
Copy link
Copy Markdown
Contributor

getsantry bot commented Sep 6, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added Stale and removed Stale labels Sep 6, 2025
@getsantry getsantry bot added the Stale label Nov 5, 2025
@getsantry getsantry bot closed this Nov 13, 2025
auto-merge was automatically disabled November 13, 2025 08:00

Pull request was closed

@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants