Skip to content

Conversation

@matthewtlam
Copy link
Contributor

@matthewtlam matthewtlam commented Feb 19, 2025

Update the P4Runtime Spec (now a .adoc). Previous discussion on #547

cc: @smolkaj, @jafingerhut

Copy link
Collaborator

@chrispsommers chrispsommers left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

Copy link
Contributor

@jafingerhut jafingerhut left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@smolkaj smolkaj left a comment

Choose a reason for hiding this comment

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

LGTM

@matthewtlam matthewtlam closed this by deleting the head repository Feb 21, 2025
@chrispsommers
Copy link
Collaborator

Hi @matthewtlam, I see you closed this, but it was on deck to be merged. Did you change your mind? Otherwise, please reopen it and we can merge it. Thanks.

@matthewtlam
Copy link
Contributor Author

Apologies to everyone who reviewed this PR. It seems that my github workspace got deleted from being inactive and I am not too sure how to recover it. Made a new PR #548

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