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

tc-egress example: fix name attribute, update dependencies #130

Closed
wants to merge 2 commits into from

Conversation

dmitris
Copy link
Contributor

@dmitris dmitris commented Aug 1, 2023

PR updates dependencies in examples/tc-egress.

@netlify
Copy link

netlify bot commented Aug 1, 2023

Deploy Preview for aya-rs ready!

Name Link
🔨 Latest commit c0bc508
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs/deploys/64c9525e9cc50b0008d4a2be
😎 Deploy Preview https://deploy-preview-130--aya-rs.netlify.app/book/programs/classifiers
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Dmitry Savintsev <[email protected]>
@dmitris dmitris changed the title tc-egress example: update dependencies tc-egress example: fix name attribute, update dependencies Aug 1, 2023
@tamird
Copy link
Member

tamird commented Aug 1, 2023

Let's drop the dependency update; it doesn't do anything because those are equivalent semvers. We use dependabot in this repository.

@dmitris dmitris closed this Aug 1, 2023
@dmitris dmitris deleted the deps-upd branch August 1, 2023 18:47
@tamird
Copy link
Member

tamird commented Aug 1, 2023

@dmitris fixing the name attribute is still useful, why not do that?

@tamird
Copy link
Member

tamird commented Aug 1, 2023

nm I'm doing this in #131.

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