Skip to content

[docs] Remove OpenLineage Cross Provider dependency documentation#63836

Open
TylerLubeck wants to merge 3 commits intoapache:mainfrom
TylerLubeck:openlineage-docs
Open

[docs] Remove OpenLineage Cross Provider dependency documentation#63836
TylerLubeck wants to merge 3 commits intoapache:mainfrom
TylerLubeck:openlineage-docs

Conversation

@TylerLubeck
Copy link

Best I can tell, this is out of date and unused:

  • The extras don't exist anymore
  • The "extra" dependencies are now full-on requirements

Was generative AI tooling used to co-author this PR?

Nope

Best I can tell, this is out of date and unused:
- The extras don't exist anymore
- The "extra" dependencies are now full-on requirements
@mobuchowski
Copy link
Contributor

I think this is autogenerated from something. @kacpermuda you might know about this

@TylerLubeck
Copy link
Author

I went spelunking and think I found where it comes from, but would love some guidance if the most recent commit doesn't do it

@kacpermuda
Copy link
Collaborator

Not sure where it comes from, but I'd guess it is auto generated from provider.yml or something similar. But given that these two libs are required by OL, it makes sense to remove them from optional. @mobuchowski maybe run full tests on this PR and see it if any check breaks (auto generation tries to modify file), if not , we should be fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants