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

Implement getExtension and getDescription for importers #1487

Merged
merged 3 commits into from
Jun 28, 2016

Conversation

zesaro
Copy link
Contributor

@zesaro zesaro commented Jun 9, 2016

Implement getExtension and getDescription for importers. #1444

  • Change in CHANGELOG.md described
  • Tests created for changes

@zesaro zesaro force-pushed the issue1444 branch 5 times, most recently from 4554cf5 to 800b247 Compare June 13, 2016 14:34
@@ -34,7 +35,7 @@
* journals, e.g., Physical Review Letters, don't use pages anymore)
*
* check here for details on the format
* http://www.ecst.csuchico.edu/~jacobsd/bib/formats/endnote.html
* http://www.ecst.csuchico.edu/~jacobsd/bib/formats/endnote.html (outdated)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you check whether there is an other source for the endnote format?

@zesaro zesaro force-pushed the issue1444 branch 2 times, most recently from 5aa80fe to 023f63c Compare June 20, 2016 11:36
@@ -37,7 +38,7 @@
* Importer for the Medline format.
*
* check here for details on the format
* --INSERT INFO HERE--
* http://www.nlm.nih.gov/bsd/mms/medlineelements.html
Copy link
Contributor

Choose a reason for hiding this comment

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

that's the wrong link, you need the link for the xml elements.

@tschechlovdev tschechlovdev added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed stupro-ready-for-internal-review labels Jun 27, 2016
@zesaro zesaro force-pushed the issue1444 branch 2 times, most recently from 023f63c to 931fb4a Compare June 27, 2016 09:40
@tschechlovdev tschechlovdev changed the title [WIP] implement getExtension and getDescription for importers Implement getExtension and getDescription for importers Jun 27, 2016
@@ -234,12 +231,13 @@ public String getFormatName() {

@Override
public List<String> getExtensions() {
return null;
//TODO:
Copy link
Member

Choose a reason for hiding this comment

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

do the todo 😄

@@ -226,17 +221,17 @@ private void readLine(BufferedReader in) throws IOException {

/**
* Read multiple lines.
*
* <p>
Copy link
Member

Choose a reason for hiding this comment

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

Why the additonal <p>?

@simonharrer
Copy link
Contributor

Please fix the minor things. After this, mark it again for ready-for-review and we will merge this in.

@simonharrer simonharrer removed the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Jun 28, 2016
@tobiasdiez tobiasdiez added this to the v3.5 milestone Jun 28, 2016
@simonharrer simonharrer merged commit 075a9ca into JabRef:master Jun 28, 2016
@zesaro zesaro deleted the issue1444 branch July 18, 2016 17:15
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.

5 participants