Skip to content

Update user_fields logic for scix id to avoid empty bibdata collisions#19

Merged
tjacovich merged 2 commits into
adsabs:mainfrom
mugdhapolimera:user-fields-update
Nov 6, 2025
Merged

Update user_fields logic for scix id to avoid empty bibdata collisions#19
tjacovich merged 2 commits into
adsabs:mainfrom
mugdhapolimera:user-fields-update

Conversation

@mugdhapolimera
Copy link
Copy Markdown
Contributor

No description provided.

@mugdhapolimera mugdhapolimera self-assigned this Nov 6, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Nov 6, 2025

Pull Request Test Coverage Report for Build 19150662626

Details

  • 22 of 23 (95.65%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.5%) to 93.811%

Changes Missing Coverage Covered Lines Changed/Added Lines %
SciXPipelineUtils/scix_id.py 22 23 95.65%
Totals Coverage Status
Change from base Build 19117418903: 0.5%
Covered Lines: 288
Relevant Lines: 307

💛 - Coveralls

@tjacovich
Copy link
Copy Markdown
Collaborator

Looks like just a couple of linter issues to resolve and this is good to merge.

@tjacovich tjacovich merged commit 7f32c56 into adsabs:main Nov 6, 2025
2 checks passed
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.

3 participants