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

Fix default failures_manifest_file #13548

Merged

Conversation

chubarovNick
Copy link
Contributor

Recently, we tried to override the failures_manifest_file configuration in our CI pipeline to achieve the partitioning of our test suite on top of the same file system.

Thanks :)

@josevalim
Copy link
Member

Thank you. Given this option does not work at all, I wonder if we should use this opportunity to rename it to _path, as that would be the more consistent name to use in Elixir. But don't worry about it, I'd rename it after merge.

@josevalim josevalim merged commit 9519a72 into elixir-lang:main May 10, 2024
9 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

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

Successfully merging this pull request may close these issues.

None yet

2 participants