-
Notifications
You must be signed in to change notification settings - Fork 121
Add journal information to citation #1045
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
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Compliance Guide 🔍Below is a summary of compliance checks for this PR:
Compliance status legend🟢 - Fully Compliant🟡 - Partial Compliant 🔴 - Not Compliant ⚪ - Requires Further Human Verification 🏷️ - Compliance label |
||||||||||||||||||||||||
| preferred-citation: | ||
| type: article | ||
| title: "MFC 5.0: An exascale many-physics flow solver" | ||
| journal: "arXiv preprint arXiv.2503.07953" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: Replace the journal field with status: "preprint" to correctly classify the work as a preprint, as arXiv is a repository and not a peer-reviewed journal. [possible issue, importance: 6]
| journal: "arXiv preprint arXiv.2503.07953" | |
| status: "preprint" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds journal information to the preferred citation for MFC 5.0. The change updates the citation metadata to include the journal field for the arXiv preprint.
- Adds journal field to the preferred citation entry
| preferred-citation: | ||
| type: article | ||
| title: "MFC 5.0: An exascale many-physics flow solver" | ||
| journal: "arXiv preprint arXiv.2503.07953" |
Copilot
AI
Nov 14, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The journal field contains a redundant arXiv identifier. The DOI field already specifies '10.48550/arXiv.2503.07953'. The journal field should be simplified to 'arXiv preprint' or formatted as 'arXiv' to avoid duplication and follow standard citation practices.
| journal: "arXiv preprint arXiv.2503.07953" | |
| journal: "arXiv preprint" |
PR Type
Documentation
Description
Add journal field to citation metadata
Specify arXiv preprint as publication venue
Diagram Walkthrough
File Walkthrough
CITATION.cff
Add journal field to citation metadataCITATION.cff
journalfield with value "arXiv preprint arXiv.2503.07953"Note
Add
journalfield topreferred-citationinCITATION.cfffor the MFC 5.0 article.Written by Cursor Bugbot for commit 067d1a5. Configure here.