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

scope: check enabled_if for private libs too #10584

Merged

Conversation

anmonteiro
Copy link
Collaborator

suggested in #10549 (comment)

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@anmonteiro anmonteiro force-pushed the anmonteiro/refactor-scope-lib_entries_of_pkg branch from 937d692 to 034e69f Compare May 26, 2024 02:10
@anmonteiro
Copy link
Collaborator Author

Merging to unblock the next PR. This is a simple refactor + checking for enabled_if in private libs

@anmonteiro anmonteiro merged commit 9e3db36 into ocaml:main May 26, 2024
25 of 28 checks passed
@anmonteiro anmonteiro deleted the anmonteiro/refactor-scope-lib_entries_of_pkg branch May 26, 2024 02:20
@anmonteiro
Copy link
Collaborator Author

in hindsight this should have gone in with #10585

MA0010 pushed a commit to MA0010/dune that referenced this pull request Jun 5, 2024
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant