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

Nockma backend: translate trace to %puts hints #3053

Merged
merged 6 commits into from
Oct 7, 2024
Merged

Conversation

lukaszcz
Copy link
Collaborator

@lukaszcz lukaszcz commented Sep 20, 2024

@lukaszcz lukaszcz added this to the 0.6.7 milestone Sep 20, 2024
@lukaszcz lukaszcz self-assigned this Sep 20, 2024
@lukaszcz lukaszcz force-pushed the nock-trace-hint branch 2 times, most recently from 2275baa to 209e81a Compare September 23, 2024 08:45
@lukaszcz lukaszcz marked this pull request as ready for review September 23, 2024 16:32
@lukaszcz lukaszcz requested review from paulcadman and janmasrovira and removed request for paulcadman September 23, 2024 16:32
@lukaszcz
Copy link
Collaborator Author

Should we merge this or wait? Because this depends on an unmerged branch in anoma

@paulcadman
Copy link
Collaborator

Should we merge this or wait? Because this depends on an unmerged branch in anoma

Good point, let me check if the unmerged branch merges cleanly into base.

@paulcadman
Copy link
Collaborator

Should we merge this or wait? Because this depends on an unmerged branch in anoma

Good point, let me check if the unmerged branch merges cleanly into base.

We should wait - because the corresponding Anoma PR does not merge cleanly after the 'defactor'.

@paulcadman
Copy link
Collaborator

The Anoma change got merged - so we can merge this now:

@lukaszcz lukaszcz merged commit 40b71b9 into main Oct 7, 2024
4 checks passed
@lukaszcz lukaszcz deleted the nock-trace-hint branch October 7, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use Anoma nock interpreter %puts hint feature in Nock backend debug mode
2 participants