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.

Sign up for GitHub

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

Jump to bottom

rss_client: fix syntax error #95

Merged
merged 1 commit into from
Jan 4, 2022
Merged

rss_client: fix syntax error #95

merged 1 commit into from
Jan 4, 2022

Conversation

yanan-fu
Copy link
Contributor

Issue:#94

Signed-off-by: Yanan Fu [email protected]

@yanan-fu
Copy link
Contributor Author

@ldoktor I found this issue was introduced by #87, could you help review ?
Thanks!

ldoktor
ldoktor approved these changes Jan 4, 2022
View reviewed changes
Copy link
Contributor

@ldoktor ldoktor left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this typo

@ldoktor ldoktor merged commit 319fcd3 into avocado-framework:master Jan 4, 2022
@ldoktor
Copy link
Contributor

ldoktor commented Jan 4, 2022

@yanan-fu would you need us to push a new version to pip or are you following the master so it'd be possible for us to wait for more fixes to accumulate?

@yanan-fu
Copy link
Contributor Author

yanan-fu commented Jan 4, 2022

@yanan-fu would you need us to push a new version to pip or are you following the master so it'd be possible for us to wait for more fixes to accumulate?

@ldoktor Master is ok for our testing, you can push new version to pip as your schedule, thanks for asking !

@yanan-fu yanan-fu deleted the rss_client branch January 20, 2022 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ldoktor ldoktor ldoktor approved these changes

@luckyh luckyh luckyh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yanan-fu @ldoktor @luckyh