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

Fix to fractional seconds trailing zeros #2029

Merged
merged 2 commits into from
Jun 29, 2024
Merged

Conversation

mereolog
Copy link
Contributor

Description

This removes the fractional second part of xsd:dataTime values as required by the spec.

Fixes: #1993

Checklist:

  • I'm familiar with the FIBO developer quide. My contribution meets all the requirements described there.
  • My contribution follows the principles of best practices for FIBO.
  • My changes have been reconciled with latest master and no merge conflicts remain.
  • This PR is related to exactly one issue. The issue is referenced by using a GitHub keyword such as "fixes", "closes", or "resolves".
  • Hygiene tests have been applied by a PR with "(WIP)" in title.
  • The issue has been tested locally using a reasoner (for ontology changes).

Signed-off-by: mereolog <pawel.garbacz@makolab.com>
@mereolog mereolog added this to the 2024Q2 FIBO Release milestone Jun 28, 2024
@mereolog mereolog self-assigned this Jun 28, 2024
Signed-off-by: mereolog <pawel.garbacz@makolab.com>
@mereolog mereolog changed the title Fix to fractional seconds trailing zeros (WIP) Fix to fractional seconds trailing zeros Jun 28, 2024
@mereolog mereolog marked this pull request as ready for review June 28, 2024 14:40
Copy link
Contributor

@ElisaKendall ElisaKendall left a comment

Choose a reason for hiding this comment

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

This looks fine to me.

@mereolog mereolog merged commit faa996a into master Jun 29, 2024
4 checks passed
@mereolog mereolog deleted the fix_to_xsddatatime_values branch June 29, 2024 11:10
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.

Wrong values of xsd:dateTime
3 participants