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

Clean export translators #1463

Merged
merged 3 commits into from Nov 7, 2017

Conversation

@zuphilip
Copy link
Collaborator

commented Nov 4, 2017

This is just some cleanup and relates to zotero/zotero-connectors#197.

zuphilip added some commits Nov 4, 2017

Delete browserSupport for export translators only
The option browserSupport has not use for export translators only,
i.e. translators with translatorType 2. This option is already
missing for
* BibLaTeX.js
* TEI.js
* Zotero RDF.js
Fix spacing in some translators headers
Found with
   grep 'translatorType":2' *.js
   grep 'translatorType":4' *.js

@adam3smith adam3smith merged commit 67eadc5 into zotero:master Nov 7, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Copy link
Collaborator

commented Nov 7, 2017

Thanks

@zuphilip zuphilip deleted the zuphilip:clean-export-translators branch Nov 7, 2017

psisquared2 added a commit to psisquared2/translators that referenced this pull request Feb 8, 2018

Clean export translators (zotero#1463)
* Delete browserSupport for export translators only

The option browserSupport has not use for export translators only,
i.e. translators with translatorType 2. This option is already
missing for
* BibLaTeX.js
* TEI.js
* Zotero RDF.js

* Fix spacing in some translators headers

Found with
   grep 'translatorType":2' *.js
   grep 'translatorType":4' *.js

* Adjust target to our standards in two translators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.