Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
STAR: Update after breaking changes to vendor export #2681
Who is this PR for?
What problem does this PR fix?
The vendor changed export formats (eg, #2526).
What does this PR do?
This makes the minimal update to the new format, and factors out the importer code for reading and math to a single file since it's now the same. This only really makes sense since the current integration is minimal and not expected to change.
Updates the tests to be integration tests, since the refactoring here didn't work with the lower-level testing strategy. Also move star importer bits into their own folder.
Which features or pages does this PR touch?
Does this PR use tests to help verify we can deploy these changes quickly and confidently?