-
Notifications
You must be signed in to change notification settings - Fork 11
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
fix: ignore initial server response. #191
Conversation
url: https://github.com/atsign-foundation/at_libraries.git | ||
path: at_lookup | ||
ref: trunk | ||
path: ../../at_lookup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to remove this dependency override after this pull request has been merged to trunk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gkc: Just a thought, Can we leave dependency overrides with path so that for subsequent pushes, at_lookup will be picked from the current branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just that function tests will run the current branch instead of trunk branch. We can catch the issue when we raise PR, instead of knowing them after we merge to trunk branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah sorry, yes of course that makes sense since it's all in the same repo and we're talking about the functional tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gkc: Thank you. So leaving the dependency_overrides as is pointing to path.
- What I did
@
- How I did it
@
prompt.- How to verify it
- Description for the changelog