traefik / yaegi

Yaegi is Another Elegant Go Interpreter
https://pkg.go.dev/github.com/traefik/yaegi
Apache License 2.0
7.02k stars 349 forks source link

interp: fix fieldName method parsing embedded + generic fields #1572

Closed zbysir closed 1 year ago

zbysir commented 1 year ago

Fix https://github.com/traefik/yaegi/issues/1571

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

ldez commented 1 year ago

Hello,

can you add a test file?

zbysir commented 1 year ago

Hi, I added one test: https://github.com/traefik/yaegi/pull/1572/commits/9b6d83c39f3a60f50d4586ea0547f125cb695453.

Is there a need for a separate test file?

ldez commented 1 year ago

It's not the right place, you have to add a test file here.

zbysir commented 1 year ago

Thanks for the tip, please review again