Skip to content

Conversation

@murataslan1
Copy link
Contributor

Summary

This PR fixes outdated examples in the openbb-apachebeam directory.

Changes

  • examples/openbb-apachebeam/README.md: Fix test-run command to point to example tests directory, fix typos and grammar
  • examples/openbb-apachebeam/requirements.txt: Add core openbb dependency required by example
  • examples/openbb-apachebeam/tests/test_obb_pipeline.py: Fix code formatting, fix 'Print nes' typo to 'Print news'

Fixes #7181

Testing

The changes are documentation and dependency fixes that don't require runtime testing.

@CLAassistant
Copy link

CLAassistant commented Dec 19, 2025

CLA assistant check
All committers have signed the CLA.

@deeleeramone
Copy link
Contributor

Thanks for the PR and spotting some typos.

examples/openbb-apachebeam/requirements.txt: Add core openbb dependency required by example

This part is not correct, openbb is NOT a dependency for running the example code. There is nothing wrong with the requirements.txt file.

- examples/openbb-apachebeam/README.md: Fix test-run command to point to example tests directory, fix typos and grammar
- examples/openbb-apachebeam/requirements.txt: Add core 'openbb' dependency required by example
- examples/openbb-apachebeam/tests/test_obb_pipeline.py: Fix code formatting, fix 'Print nes' typo to 'Print news'

Fixes OpenBB-finance#7181
@murataslan1 murataslan1 force-pushed the fix/outdated-examples-7181 branch from 6fae89c to c2e301f Compare December 21, 2025 18:28
@murataslan1
Copy link
Contributor Author

@deeleeramone Thanks for the feedback! I'll revert the requirements.txt change. Should I do that in a new commit?

@murataslan1
Copy link
Contributor Author

@deeleeramone Done! I've reverted the requirements.txt change as suggested. The new commit only includes:

  • README.md fixes (test command path, typos)
  • Test file typo fix ('Print nes' -> 'Print news')

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.

Detected outdated examples - fix included

3 participants