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.

Sign up for GitHub

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

Jump to bottom

pm add preferred name attributes #57

Merged
merged 9 commits into from
Dec 16, 2024
Merged

pm add preferred name attributes #57

merged 9 commits into from
Dec 16, 2024

Conversation

paulkm
Copy link
Contributor

@paulkm paulkm commented Dec 16, 2024

  • add RepositoryTimeStamp attribute
  • pycodestyle fix
  • fix linter error and person test model
  • handle the test correctly
  • Update example JSON with RepositoryTimeStamp
  • Added preferred names into person JSON data

@coveralls
Copy link

Coverage Status

coverage: 99.39% (+0.002%) from 99.388%
when pulling e36ad28 on pm_add_preferred_name_attributes
into 6d569e5 on develop.

@paulkm paulkm requested a review from jlaney December 16, 2024 17:50
@paulkm paulkm merged commit 21a7453 into develop Dec 16, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlaney jlaney jlaney approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@paulkm @coveralls @jlaney