bazel-xcode / PodToBUILD

An easy way to integrate CocoaPods into Bazel
Apache License 2.0
323 stars 69 forks source link

Simplify globs, simplfy rendering #154

Closed jerrymarino closed 4 years ago

jerrymarino commented 4 years ago

This PR simplifies GlobNodes upon allocation reduce the complexity of the glob at runtime algorithms and visually.

Logically, the output is the same, but it cleans up build files where possible, especially in the case of combining subspec deps, arc, and exclusions.

Additionally, it adds a few cleanups and removed the default argument for exclude_directories as it's unneeded

This was pointed out in https://github.com/pinterest/PodToBUILD/pull/153 and submitted as a follow up for simplicity

jparise commented 4 years ago

Much nicer!