Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BAR ThaleMine - 7: go-annotation loader #2240

Merged
merged 2 commits into from
May 18, 2020

Conversation

asherpasha
Copy link
Contributor

@asherpasha asherpasha commented Apr 10, 2020

Details

This pull request makes go-annotation loader work on ThaleMine.

Testing

We have tested it on ThaleMine dev and live versions.

Checklist

Before your pull request can be approved, be sure to check all boxes:

  • Passing unit test for new or updated code (if applicable)
  • Passes all tests – according to Travis
  • Documentation (if applicable)
  • Single purpose
  • Detailed commit messages
  • Well commented code
  • Checkstyle

@danielabutano danielabutano added this to the InterMine 4.2.0 milestone May 18, 2020
@danielabutano danielabutano added this to InterMine 4.2.0 (Spring 2020) in Roadmap & release planning May 18, 2020
@danielabutano danielabutano self-requested a review May 18, 2020 13:29
Copy link
Member

@danielabutano danielabutano left a comment

Choose a reason for hiding this comment

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

Hi @asherpasha thanks for the PR! I was wondering what the Trigger (empty) commit is...just a way to restart the build?

@asherpasha
Copy link
Contributor Author

Yes, I had to push an empty commit to restart Travis. It was saying built failed.

@danielabutano danielabutano merged commit 1e9b6bf into intermine:dev May 18, 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.

None yet

2 participants