Skip to content

Global spec temp directory setup#63

Open
dpep wants to merge 2 commits intorubyatscale:mainfrom
dpep:global-spec-temp-dir
Open

Global spec temp directory setup#63
dpep wants to merge 2 commits intorubyatscale:mainfrom
dpep:global-spec-temp-dir

Conversation

@dpep
Copy link
Copy Markdown

@dpep dpep commented Mar 27, 2026

Summary

  • Move temp directory setup/cleanup to spec_helper.rb using Dir.mktmpdir block form
  • Automatically cleans up temp directories after each spec (no leaks)
  • Removes duplicate setup from singed_spec.rb and spec/support/sidekiq.rb

🤖 Generated with Claude Code

Use Dir.mktmpdir block form in spec_helper to automatically create and
cleanup temp directories for all specs. Removes duplicate setup from
individual spec files.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@dpep dpep requested a review from a team as a code owner March 27, 2026 23:44
@github-project-automation github-project-automation bot moved this to Triage in Modularity Mar 27, 2026
@dpep dpep mentioned this pull request Mar 28, 2026
Copy link
Copy Markdown
Contributor

@dduugg dduugg left a comment

Choose a reason for hiding this comment

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

It would probably be better to restore the output directory, if you feel inspired, but this is fine as is. Thanks for your contribution!

More robust and future-proof to restore the previous state
rather than destroying it.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@dpep
Copy link
Copy Markdown
Author

dpep commented Mar 28, 2026

@dduugg iterated! gtg here or would you like to iterate on anything else?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants