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

Ensure eval call knows its line numbers etc #426

Merged

Conversation

olleolleolle
Copy link

This PR adds binding and line number information to an eval'd piece of code.

Copy link

@marvindanig marvindanig left a comment

Choose a reason for hiding this comment

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

Neat.

@marvindanig
Copy link

Pinging @kjvarga to have some of these pending PRs merged asap. ☝🏼

@n-rodriguez
Copy link
Collaborator

Hi there! Any news? ping @kjvarga

@n-rodriguez
Copy link
Collaborator

@olleolleolle hello! can you please rebase your branch? Thank you!

Use #each_key, add a separating newline, and meaningfully name heredoc markers.
@olleolleolle olleolleolle force-pushed the add-line-number-to-eval-call branch from 08d1a43 to 4d93f64 Compare November 4, 2024 16:29
@olleolleolle
Copy link
Author

@n-rodriguez Now rebased!

@olleolleolle
Copy link
Author

I guess the CI configuration has to be unconfigured in the Settings page of this project, so that there are no requirements on CircleCI having finalized.

@kjvarga
Copy link
Owner

kjvarga commented Nov 4, 2024

@olleolleolle @n-rodriguez I've removed the CircleCI Webhook from settings so it shouldn't show up in the status checks. But looks like we might have to push an empty commit to clear out the errored job.

@n-rodriguez n-rodriguez merged commit efc5c86 into kjvarga:master Nov 4, 2024
56 of 57 checks passed
@n-rodriguez
Copy link
Collaborator

@olleolleolle @kjvarga Thank you!

@olleolleolle olleolleolle deleted the add-line-number-to-eval-call branch November 5, 2024 07:46
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.

4 participants