Skip to content

Conversation

@majocha
Copy link
Contributor

@majocha majocha commented Sep 18, 2024

Since the tests run in temp directories by default, this code in single-test.fs should no longer be needed.

Let's see if it is.

@majocha majocha requested a review from a team as a code owner September 18, 2024 05:40
@github-actions
Copy link
Contributor

✅ No release notes required

@KevinRansom
Copy link
Contributor

Yeah!!! the tests are probably not as consistent as you think. Different tests, even within the ComponentTests suite have different strategies ... usually depending on how the test originally work.

@majocha
Copy link
Contributor Author

majocha commented Sep 18, 2024

I think I had that special case handled for that helloworld thing. I'll look into it, as this change works in my experiment PR.

@majocha
Copy link
Contributor Author

majocha commented Sep 18, 2024

Yes it's like geological strata, sediments upon sediments. But it seems this worked :)

@psfinaki
Copy link
Contributor

Yeah. We are doing geological programming this week :)

Consider conducting some smoke testing from whatever applies in this comment to a sibling PR - among other things, doing this will just help us "compile" some sanity checks for these sorts of PRs.

@majocha
Copy link
Contributor Author

majocha commented Sep 28, 2024

Subsumed by #17815

@majocha majocha closed this Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants