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

Create plant-genetic-resources-characterization-and-utilization.csl #4256

Merged
merged 3 commits into from Aug 21, 2019

Conversation

@csl-bot

This comment has been minimized.

Copy link

commented Aug 19, 2019

Awesome! You just created a pull request to the Citation Styles Language styles repository. One of our human volunteers will try to get in touch soon (usually within a week). In the meantime, I will run some automated checks. You should be notified of the results in a few minutes.

If you haven't done so yet, please make sure your style validates and follows all our other Style Requirements.

To update this pull request, visit the "Files changed" tab above, and click on the pencil icon (see below) in the top-right corner of your style to start editing.

If you have any questions, please leave a comment and we'll get back to you. While we usually respond in English, feel free to write in whatever language you're most comfortable.

@csl-bot

This comment has been minimized.

Copy link

commented Aug 19, 2019

😃 Your submission passed all our automated tests.

Below are some sample citations generated based on your proposed changes:

plant-genetic-resources-characterization-and-utilization.csl (new)
(Hancké et al., 2007; CSL search by example, 2012)
(Mares, 2001; Fenner et al., 2019)

CSL search by example (2012) CSL search by example. Available at http://editor.citationstyles.org/searchByExample/ (accessed 15 December 2012).
Fenner M et al. (2019) A data citation roadmap for scholarly data repositories. Scientific Data 6: 28.
Hancké B, Rhodes M and Thatcher M (eds) (2007) Beyond Varieties of Capitalism : Conflict, Contradiction, and Complementarities in the European Economy. Oxford University Press: Oxford and New York.
Mares I (2001) Firms and the welfare state: When, why, and how does social policy matter to employers? In: Hall PA and Soskice D (eds). Varieties of Capitalism. The Institutional Foundations of Comparative Advantage. Oxford University Press: New York, pp. 184–213.
@adam3smith
Copy link
Member

left a comment

Some nits -- mainly clean-up from the very specific content of the template.

</if>
<else>
<choose>
<if variable="call-number">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

that seems very odd -- I assume inherited? But would remove.

<if type="bill book graphic legal_case legislation motion_picture song" match="any">
<text variable="title" font-style="italic" text-case="title"/>
</if>
<else-if type="report" match="any">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

not needed - remove.

</macro>
<macro name="access-date">
<choose>
<if type="webpage post post-weblog">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

add match="any"

</else>
</choose>
</macro>
<macro name="extra">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

What is this for? Better to remove -- I prefer not to have note/extra in styles.

</if>
</choose>
</macro>
<macro name="un-sales">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

this seems like a leftover - remove

<else-if type="bill book chapter graphic legal_case legislation motion_picture paper-conference song" match="any">
<choose>
<if variable="collection-title">
<group delimiter=", ">

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

handle the spaces here more elegantly (e.g. no need for the one before "No." and the one before "Vol." should probably be a group prefix)

</choose>
</macro>
<macro name="citation-locator">
<group>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

no need for this -- just delete macro

<text macro="author-short"/>
<text macro="year-date"/>
</group>
<text macro="citation-locator"/>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

and then handle the colon before the locator here.

<date variable="issued">
<date-part name="month" prefix=" "/>
</date>
<text variable="URL" prefix=". Available at " font-style="normal"/>

This comment has been minimized.

Copy link
@adam3smith

adam3smith Aug 20, 2019

Member

could we pack this into a macro and handle available at with a term?

Update plant-genetic-resources-characterization-and-utilization.csl
Thank you for the thorough review. Obviously not an ideal style to base this one on.
@csl-bot

This comment has been minimized.

Copy link

commented Aug 20, 2019

😟 There are some issues with your submission.

1 test failed

plant-genetic-resources-characterization-and-utilization: must define all macros that are called by <text/> and <key/> elements

expected #has_key?("citation-locator") to return true, got false

Please check the test report for details.

@csl-bot

This comment has been minimized.

Copy link

commented Aug 20, 2019

😃 Your submission passed all our automated tests.

Below are some sample citations generated based on your proposed changes:

plant-genetic-resources-characterization-and-utilization.csl (new)
(Hancké et al., 2007; CSL search by example, 2012)
(Mares, 2001; Fenner et al., 2019)

CSL search by example (2012) Citation Style Editor. See http://editor.citationstyles.org/searchByExample/ (accessed 15 December 2012).
Fenner M et al. (2019) A data citation roadmap for scholarly data repositories. Scientific Data 6: 28.
Hancké B, Rhodes M and Thatcher M (eds) (2007) Beyond Varieties of Capitalism : Conflict, Contradiction, and Complementarities in the European Economy. Oxford University Press: Oxford and New York.
Mares I (2001) Firms and the welfare state: When, why, and how does social policy matter to employers? In: Hall PA and Soskice D (eds). Varieties of Capitalism. The Institutional Foundations of Comparative Advantage. Oxford University Press: New York, pp. 184–213.

@adam3smith adam3smith merged commit 51c0851 into citation-style-language:master Aug 21, 2019

1 check passed

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

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

Great thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.