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

Document Julian year #318

Merged
merged 1 commit into from
Jun 25, 2022
Merged

Conversation

triallax
Copy link
Collaborator

No description provided.

Comment on lines +98 to +99
| [Gregorian year](https://en.wikipedia.org/wiki/Gregorian_year) | `years`, `year` |
| [Julian year](https://en.wikipedia.org/wiki/Julian_year_(astronomy)) | `julianYears`, `julianYear` |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if we should explicitly mention the length of the years in the table. One can easily check the length by going to the Wikipedia pages linked here though.

Also, should we keep the unit name as "Year" instead of "Gregorian year"? I think the latter is clearer.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if we should explicitly mention the length of the years in the table. One can easily check the length by going to the Wikipedia pages linked here though.

Yeah, we don't do anything like this for the other units either.

Also, should we keep the unit name as "Year" instead of "Gregorian year"?

👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I'm not exactly clear on what your opinions on the two issues here are. Are you saying the PR is good as-is?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, exactly. I like that we have "year" in the documentation. And I think it's okay if we don't mention the precise distinction between the two in the documentation.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be clear, by "unit name" I meant the name in the first column of the table. Also, I made the unit name in the table "Gregorian year", not "year", is that what you want?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be clear, by "unit name" I meant the name in the first column of the table.

Oh, my bad. I misunderstood your initial post. I'm okay with leaving "Gregorian year" here. Maybe we simply write "(Gregorian) year"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I don't think the parentheses are needed, just "Gregorian year" is okay in my opinion.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright.

@sharkdp sharkdp merged commit 3f5be42 into sharkdp:master Jun 25, 2022
@triallax triallax deleted the document-julian-year branch June 25, 2022 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants