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

Add missing abbreviated journal names #6292

Merged
merged 11 commits into from
Apr 16, 2020

Conversation

mayrmt
Copy link
Contributor

@mayrmt mayrmt commented Apr 15, 2020

Add more abbreviated journal names. Details:

Also fix some typos in existing journal abbreviations.

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

@mayrmt
Copy link
Contributor Author

mayrmt commented Apr 15, 2020

I don't understand, why coverage is supposedly going down. According to the codecov details, this PR only changes files, that are ignored by the code coverage tool.

I'd appreciate any insight and help. Thanks!

@koppor
Copy link
Member

koppor commented Apr 15, 2020

CodeCov decreased, because the SpringerFetcher failed this time:
grafik. Think, we have to reconfigure CodeCov to test "only" the "local" functionality - and not the one relying on external life services (AKA integration test).

Update: Not that easily possible, but I tried 01189ef.

@koppor
Copy link
Member

koppor commented Apr 15, 2020

@jlaehne You have deep knowledge about the abbreviatins (refs #5749). Could you guide @mayrmt here? Should he also add the abbreviations somewhere at https://github.com/JabRef/abbrv.jabref.org/?

From my side, I would just merge. However, I fear that the abbreviations get lost if we continue working on #5749

@@ -450,6 +450,7 @@ Advanced Materials;Adv. Mater.;;
Advanced Materials and Processes;Adv. Mater. Processes;;
Advanced Materials for Optics and Electronics;Adv. Mater. Opt. Electron.;;
Advanced Materials Forum;Adv. Mater. Forum;;
Advanced Modeling and Simulation in Engineering Sciences;Adv. Model. and Simul. in Eng. Sci.;;
Copy link
Contributor

Choose a reason for hiding this comment

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

and or in are left out in the ISO specs, so I would expect Adv. Model. Simul. Eng. Sci.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jlaehne You're right. I will fix this.

@jlaehne
Copy link
Contributor

jlaehne commented Apr 15, 2020

@mayrmt, could you add your abbreviations/corrections to https://github.com/JabRef/abbrv.jabref.org/blob/master/journals/journal_abbreviations_ams.csv

and verify that, if present, they are also correct in https://github.com/JabRef/abbrv.jabref.org/blob/master/journals/journal_abbreviations_general.csv
(has higher priority over the other lists)

As @koppor indicated, with #5749, corrections will be automatically sourced from the lists at JabRef/abbrv.jabref.org and, if just merged into the overall list at the moment, your corrections would be lost in the future.

@mayrmt
Copy link
Contributor Author

mayrmt commented Apr 15, 2020

@jlaehne I've removed some "and" from abbreviated names. I'm happy to squash these 11 commits into one, if desired. Just let me know.

Furthermore, JabRef/abbrv.jabref.org#61 mirrors these changes.

@jlaehne
Copy link
Contributor

jlaehne commented Apr 15, 2020

Thanks for the mirroring

@koppor
Copy link
Member

koppor commented Apr 16, 2020

I'll use GitHub's "Squash and merge" functionality. Thank you for the Update!

@koppor koppor merged commit 4a9247b into JabRef:master Apr 16, 2020
@mayrmt mayrmt deleted the add-abbrev-journal-names branch April 16, 2020 20:38
dextep pushed a commit to dextep/jabref that referenced this pull request Apr 17, 2020
dextep added a commit to dextep/jabref that referenced this pull request Apr 17, 2020
Siedlerchr added a commit that referenced this pull request Apr 17, 2020
…ionCaseInsensitive

* upstream/master: (25 commits)
  ActionHelper to test for present file (#6151)
  Reduce memory footprint (#6298)
  Add missing abbreviated journal names (#6292)
  fix l10n again
  fix checkstyle
  fix l10n
  Try to minimize CodeCov "wrong metrics"
  Showing correct icon on main table linked files column (#6264)
  Fix labels for outdated dependency issue
  Change one more line
  Squashed 'src/main/resources/csl-styles/' changes from c31d9ca..c1793d2
  Resolve unit test from failing
  Add one more change
  Fix errors
  RIS import takes the wrong date and duplicates abstract (#6272)
  Update EntryTypeView.java
  Change to the old school format
  Fix XmpExporterTest (#6289)
  Add checkstyle screenshot (and lint guidelines-...md)
  Squashed 'src/main/resources/csl-styles/' changes from db54e56..c31d9ca
  ...
dextep added a commit to dextep/jabref that referenced this pull request May 3, 2020
koppor pushed a commit that referenced this pull request Jan 1, 2023
43566f2 Update molecular-oncology.csl (#6354)
38f2b5f Update san-francisco-estuary-and-watershed-science.csl (#6350)
724cb12 Update australasian-journal-of-philosophy.csl (#6344)
df6af86 Fix webpage in-text citation for council-of-science-editors-author-date.csl (#6318)
6900b58 Add month to magazine for Bluebook
e0a8148 Update cambridge-university-press-author-date-cambridge-a.csl (#6345)
0823448 Add space & comma before pp (#6343)
ff38cd2 Update american-journal-of-respiratory-and-critical-care-medicine.csl (#6341)
8727dfb Update early-music-history.csl (#6323)
1154354 Update boletin-de-pediatria.csl (#6310)
f25438e Update pravnik.csl (#6309)
db7a4ae Update zoological-journal-of-the-linnean-society.csl (#6304)
6c043a7 Create polygraphia.csl (#6307)
255e00c Create intellect-newgen-books.csl (#6308)
323629f Update historical-materialism.csl (#6300)
f62b70d Create european-review-of-international-studies.csl (#6301)
dff2698 Update ucl-university-college-harvard.csl (#6298)
0ce09c9 Update sciences-po-ecole-doctorale-note-french.csl (#6290)
bdd53ec Update sciences-po-ecole-doctorale-author-date.csl (#6291)
efde4d4 Create journal-of-law-medicine-ethics.csl (#6296)
7539b2c Create theses-de-sorbonne-universite.csl (#6295)
905f25a Update biochemical-society-transactions.csl (#6292)
a76a3f5 Update smithsonian-institution-scholarly-press author-date and note (#6294)
e6b6c6c Create exploration-of-targeted-anti-tumor-therapy.csl (#6276)
024c9c8 Create nys-nydanske-studier.csl (#6331)
d9ac8e1 Update vox-sanguinis.csl (#6327)
9a98e92 Remove DOI from Genetics & Molecular Biology

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: 43566f2
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