-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
ct: Remove documentation for deprecated ?config #7825
ct: Remove documentation for deprecated ?config #7825
Conversation
CT Test ResultsNo tests were run for this PR. This is either because the build failed, or the PR is based on a branch without GH actions tests configured. Results for commit 929a906 To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass. See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally. Artifacts// Erlang/OTP Github Action Bot |
maybe let's treat this as a bug fix and target it to maint branch. |
3085139
to
e0bd674
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please squash commits as change is small and separate commits will not be worth having after review.
Per dcda9b5 and review of erlang#7806, usage of CT's `?config` macro is no longer recommended. Replace the documentation referencing it with `proplists:get_value`, which works the same way and can take an optional third argument for the default return value.
710df91
to
929a906
Compare
Per dcda9b5 and review of #7806, usage of CT's
?config
macro is no longer recommended. Replace the documentation referencing it withproplists:get_value
, which works the same way and can take an optional third argument for the default return value.