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

Update case.ex with correct :since for register_test/6 #13642

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

vanderhoop
Copy link
Contributor

while addressing the ExUnit.Case.register_test/3 deprecation in elixir-wallaby/wallaby#776, I wondered how long register_test/6 had been around. In looking at the source and the docs, I uncovered what I believe is an incorrect since attribute

while addressing the `ExUnit.Case.register_test/3` deprecation in elixir-wallaby/wallaby#776, I wondered how long `register_test/6` had been around. In looking at the source and the docs, I uncovered what I believe is an incorrect `since` attribute
@@ -578,7 +578,7 @@ defmodule ExUnit.Case do
display. You can use `ExUnit.plural_rule/2` to set a custom
pluralization.
"""
@doc since: "1.10.0"
@doc since: "1.11.0"
Copy link
Contributor Author

@vanderhoop vanderhoop Jun 6, 2024

Choose a reason for hiding this comment

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

I initially looked through this file's history by version tag to check this. It would have been faster had I just searched the changelogs 😄

https://github.com/elixir-lang/elixir/blob/v1.11/CHANGELOG.md#exunit-1

@josevalim josevalim merged commit 9ffe447 into elixir-lang:v1.17 Jun 6, 2024
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

josevalim pushed a commit that referenced this pull request Jun 6, 2024
while addressing the `ExUnit.Case.register_test/3` deprecation in elixir-wallaby/wallaby#776, I wondered how long `register_test/6` had been around. In looking at the source and the docs, I uncovered what I believe is an incorrect `since` attribute
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.

3 participants