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

Update circulation-journal.csl #3197

Merged
merged 1 commit into from Dec 21, 2017

Conversation

Projects
None yet
3 participants
@naritatsu
Copy link
Contributor

naritatsu commented Dec 18, 2017

I changed line 2 : page-range-format="minimal" to page-range-format="expanded" , according to the journal citation style.

Update circulation-journal.csl
I changed line 2 : page-range-format="minimal" to page-range-format="expanded" , according to the journal citation style.
@csl-bot

This comment has been minimized.

Copy link

csl-bot commented Dec 18, 2017

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.

image

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

csl-bot commented Dec 18, 2017

😃 Your submission passed all our automated tests.

@naritatsu

This comment has been minimized.

Copy link
Contributor Author

naritatsu commented Dec 21, 2017

Do I need to wait until this change is reflected?

@rmzelle rmzelle merged commit 8c33e1c into citation-style-language:master Dec 21, 2017

1 check passed

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

This comment has been minimized.

Copy link
Member

rmzelle commented Dec 21, 2017

Thanks! (it just takes us a while sometimes to respond)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.