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

porousflow JOSS paper: minor mods #15591

Closed
WilkAndy opened this issue Jul 11, 2020 · 1 comment
Closed

porousflow JOSS paper: minor mods #15591

WilkAndy opened this issue Jul 11, 2020 · 1 comment
Labels
P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.

Comments

@WilkAndy
Copy link
Contributor

Reason

Need to make minor mods to the PorousFlow JOSS paper

Design

https://doi.org/10.1016/j.softx.2020.100430 is INVALID because of 'https://doi.org/' prefix

While you're at it, could you avoid repeating the repository URL (which already exists in the sidebar)? Also, it's a bit unusual to launch into a bullet list of applications after only one short sentence saying what the package does. I suppose I don't object to the present organization, but an EiC might.

Impact

@WilkAndy WilkAndy added P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software. labels Jul 11, 2020
WilkAndy added a commit to WilkAndy/moose that referenced this issue Jul 11, 2020
WilkAndy added a commit to WilkAndy/moose that referenced this issue Jul 11, 2020
WilkAndy added a commit to WilkAndy/moose that referenced this issue Jul 11, 2020
milljm pushed a commit to milljm/moose that referenced this issue Jul 15, 2020
@loganharbour
Copy link
Member

Closed in #15592

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.
Projects
None yet
Development

No branches or pull requests

2 participants