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

generate: fix multiple Go file type parsing bug #4507

Merged
merged 1 commit into from Feb 11, 2021

Conversation

estroz
Copy link
Member

@estroz estroz commented Feb 11, 2021

Description of the change:

  • internal/generate/clusterserviceversion/bases/definitions: consider both untyped and Typ-typed *ast.Object's that underly an *ast.Ident, such that types in other files are found properly

Motivation for the change: see #3744

Closes #3744

/kind bug

Signed-off-by: Eric Stroczynski ericstroczynski@gmail.com

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Feb 11, 2021
@estroz
Copy link
Member Author

estroz commented Feb 11, 2021

/cc @jmrodri
/ping @IBMRob

untyped and Typ-typed *ast.Object's that underly an *ast.Ident,
such that types in other files are found properly

Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

the tests helped me understand what was going on. +1

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 11, 2021
@estroz estroz merged commit f998376 into operator-framework:master Feb 11, 2021
@estroz estroz deleted the bugfix/3744 branch February 11, 2021 02:10
@estroz
Copy link
Member Author

estroz commented Feb 11, 2021

/cherry-pick v1.3.x

@openshift-cherrypick-robot

@estroz: new pull request created: #4508

In response to this:

/cherry-pick v1.3.x

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@estroz
Copy link
Member Author

estroz commented Feb 11, 2021

/cherry-pick v1.4.x

@openshift-cherrypick-robot

@estroz: new pull request created: #4509

In response to this:

/cherry-pick v1.4.x

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSV Field Markers are not collected if they are in another go file
4 participants