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

fix(describe-pp): show output on successive runs #10340

Merged
merged 2 commits into from
Apr 1, 2024

Conversation

anmonteiro
Copy link
Collaborator

@anmonteiro anmonteiro commented Mar 30, 2024

@anmonteiro anmonteiro requested a review from rgrinberg March 30, 2024 04:25
@anmonteiro anmonteiro force-pushed the anmonteiro/describe-pp-force branch from 3a6e012 to e0fff6e Compare March 30, 2024 04:25
Copy link
Member

@rgrinberg rgrinberg left a comment

Choose a reason for hiding this comment

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

I suppose this is good enough for a hot fix

@anmonteiro anmonteiro force-pushed the anmonteiro/describe-pp-force branch from e0fff6e to 8ecd748 Compare March 30, 2024 22:22
@anmonteiro
Copy link
Collaborator Author

@rgrinberg I pushed a version that uses Action_exec.exec. Looks a bit better IMO

@anmonteiro anmonteiro force-pushed the anmonteiro/describe-pp-force branch from 8ecd748 to 4b75925 Compare March 31, 2024 21:22
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@anmonteiro anmonteiro force-pushed the anmonteiro/describe-pp-force branch from 4b75925 to 70f28f9 Compare April 1, 2024 19:53
@anmonteiro anmonteiro requested a review from rgrinberg April 1, 2024 19:54
@anmonteiro anmonteiro merged commit 5a2ec99 into ocaml:main Apr 1, 2024
24 of 27 checks passed
@anmonteiro anmonteiro deleted the anmonteiro/describe-pp-force branch April 1, 2024 19:58
pmwhite pushed a commit to pmwhite/dune that referenced this pull request Apr 2, 2024
* fix(describe-pp): show output on successive runs

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>

* code-review: use Action_exec.exec instead

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>

---------

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.

2 participants