Skip to content

Conversation

@JordanHood
Copy link
Member

What does this PR do? (please provide any background)

There's some code in tapestry-exodus causing a crash as the functionallity no longer existes, exodus should have been removed a while ago, so this just culls it from the-works

What tests does this PR have?

How can this be tested?

Any tech debt?

Screenshots / Screencast

What gif best describes how you feel about this work?

By approving a review you are confirming you have...

  • Witnessed the work behaving as expected (this could be on the author's machine or screencast).
  • Checked for coding anti-patterns.
  • Checked for appropriate test coverage.
  • Checked all the tests are passing.

Copy link
Contributor

@david-r-edgar david-r-edgar left a comment

Choose a reason for hiding this comment

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

looks good, haven't tested locally

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.

3 participants