Skip to content

reflect: implement NewAt - #5812

Open
jakebailey wants to merge 1 commit into
tinygo-org:devfrom
jakebailey:reflect-new-at
Open

jakebailey wants to merge 1 commit into
tinygo-org:devfrom
jakebailey:reflect-new-at

Conversation

@jakebailey

Copy link
Copy Markdown
Member

This was missing and is pretty easy to do. Unblocks some upstream stuff (eventually...)

@dgryski dgryski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@davecheney davecheney left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@jakebailey I tried to send a PR to add davecheney@2e1a22b but your fork isn't configured for PRs. Feel free to cherry pick that if you like

Co-authored-by: Dave Cheney <dave@cheney.net>
@jakebailey

Copy link
Copy Markdown
Member Author

Yeah, I figured it wasn't worth having that open 😄

Added your code and Co-authored-by'd it.

@davecheney davecheney left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. let me know if you prefer a different way of sending patches. The PR on PR idea I shamelessly stole from @proppy.

@jakebailey

Copy link
Copy Markdown
Member Author

I'm avoiding it mainly because I'm using jj so external changes coming into my own changes is a little funky, but I might be able to figure it out.

@davecheney davecheney left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Thank you

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.

3 participants