Skip to content

remove './gradlew tomcatRun' - it's both un-used and appears totally broken anyway - #795

Closed
vorburger wants to merge 1 commit into
apache:developfrom
vorburger:rm-tomcatRun
Closed

vorburger wants to merge 1 commit into
apache:developfrom
vorburger:rm-tomcatRun

Conversation

@vorburger

Copy link
Copy Markdown
Member

No description provided.

@maektwain

Copy link
Copy Markdown

This test also failed, are we going to use @ignore again, seems like this is more consistent now.

@vorburger

Copy link
Copy Markdown
Member Author

I'm converting this to Draft, because I created it re. https://markmail.org/thread/jnlerankgfbq6zhe, but it looks like Petri Tuomola may actually use tomcatRun for FINERACT-730, so let's hold this.

PS: @maektwain Yes, as per https://github.com/apache/fineract#pull-requests. Unrelated to this PR.

@vorburger
vorburger marked this pull request as draft May 2, 2020 16:52
@vorburger

vorburger commented May 5, 2020 •

Copy link
Copy Markdown
Member Author

Closing this one, because both tomcatRun and tomcatRunWAR are (about to be) removed in #807.

@ptuomola

@vorburger vorburger closed this May 5, 2020
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.

2 participants