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

ct: Remove long deprecated support for sequences/0 callback #9554

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jchristgit
Copy link
Contributor

This has printed a deprecation warning for as long as my git history goes back (see diff in ct_util). I have done a code search on GitHub 1 and with the exception of the OTP repository the only repositories which had anything specified in here were last updated in 2011.

Spotted this while working on #9530 and figured based on the comment this should be safe to remove.

This has printed a deprecation warning for as long as my git history
goes back (see diff in `ct_util`). I have done a code search on GitHub
[1] and with the exception of the OTP repository the only repositories
which had anything specified in here were last updated in 2011.

[1]: https://github.com/search?q=language%3Aerlang+%2F%5Esequences.*%5Cn.*%5C%5B.%2B%5C%5D%2F+path%3A*_SUITE.erl+&type=code&p=1
Copy link
Contributor

github-actions bot commented Mar 6, 2025

CT Test Results

  2 files   57 suites   1h 16m 45s ⏱️
451 tests 438 ✅ 13 💤 0 ❌
485 runs  469 ✅ 16 💤 0 ❌

Results for commit 87ea2d1.

♻️ This comment has been updated with latest results.

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

@rickard-green rickard-green added the team:PS Assigned to OTP team PS label Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:medium team:PS Assigned to OTP team PS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants