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

supervisor: export sup_ref type #6893

Closed
wants to merge 1 commit into from

Conversation

tsloughter
Copy link
Contributor

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2023

CT Test Results

       2 files       88 suites   41m 3s ⏱️
1 863 tests 1 814 ✔️ 48 💤 1
2 154 runs  2 103 ✔️ 50 💤 1

For more details on these failures, see this check.

Results for commit 2124f6f.

♻️ 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 Feb 20, 2023
@IngelaAndin
Copy link
Contributor

We will look into this, we of course want API types to be available but in the same time want to be causes in exporting old types so that we do not lock ourselves into corners by making type that is sort of opaque but still can be matched too specified so that the user start having the wrong expectation. So that said it might take some time for us to have time to make the good decision.

@tsloughter
Copy link
Contributor Author

OK, thanks 👍

@kikofernandez
Copy link
Contributor

Sorry for the delay in the answer. I am trying to see if instead of simply exporting this type, I can also unify the types of supervisor, gen_server, gen_event, and gen_statem.

@kikofernandez
Copy link
Contributor

Super-seeded by #7205

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:PS Assigned to OTP team PS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants