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

minor date-time fixes for 1.81 release #219

Merged
merged 6 commits into from
Oct 31, 2022
Merged

minor date-time fixes for 1.81 release #219

merged 6 commits into from
Oct 31, 2022

Conversation

JeffGarland
Copy link
Collaborator

No description provided.

Quinnsicle and others added 6 commits July 2, 2022 10:41
Co-authored-by: Quinn O'Connor <qoconnor@fastenal.com>
#214)

GCC up to at least 10.2 fail to resolve
gregorian::date::date(special_values) as constexpr function due to
assignment to *this within the constructor. Refactoring constructor to
initialize the instance once leads to large performance improvement.
* to_iso_*string() use "." as fractional separator

The fractional separator for the various to_iso_* methods
is "." not "," (per to_iso_string_type() implementation).
Fix the documentation to match the implementation.

* use "ISO 8601" not "iso" in documentation

The standard is "ISO 8601", so use that instead
of just "iso" or "ISO" in comments and documentation.

* fractional seconds only included if non-zero (#110)

Consistently document that the fractional seconds
are only included if non-zero.

Use "where fffffffff" not "were fff".
GitHub Action is sunsetting the macOS 10.15 Actions runner. It will stop working intermittently until being completely removed by 2022-8-30: https://github.blog/changelog/2022-07-20-github-actions-the-macos-10-15-actions-runner-image-is-being-deprecated-and-will-be-removed-by-8-30-22
@codecov
Copy link

codecov bot commented Oct 29, 2022

Codecov Report

Merging #219 (3971490) into master (f972dc9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #219   +/-   ##
=======================================
  Coverage   91.74%   91.75%           
=======================================
  Files          77       77           
  Lines        4823     4827    +4     
=======================================
+ Hits         4425     4429    +4     
  Misses        398      398           
Impacted Files Coverage Δ
include/boost/date_time/date.hpp 96.49% <ø> (ø)
include/boost/date_time/gregorian/formatters.hpp 100.00% <ø> (ø)
include/boost/date_time/gregorian/parsers.hpp 95.23% <ø> (ø)
include/boost/date_time/gregorian_calendar.ipp 98.66% <ø> (ø)
include/boost/date_time/iso_format.hpp 100.00% <ø> (ø)
...ude/boost/date_time/local_time/local_date_time.hpp 93.03% <ø> (ø)
...ude/boost/date_time/posix_time/time_formatters.hpp 85.56% <ø> (ø)
include/boost/date_time/time_facet.hpp 85.47% <ø> (ø)
include/boost/date_time/gregorian/greg_date.hpp 100.00% <100.00%> (ø)
include/boost/date_time/special_values_parser.hpp 100.00% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f972dc9...3971490. Read the comment docs.

@JeffGarland JeffGarland merged commit 85e637c into master Oct 31, 2022
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

7 participants