Skip to content

Conversation

@forman
Copy link
Member

@forman forman commented May 28, 2024

The function zappend.api.zappend() now returns the number of slices processed.

Closes #93

Checklist (strike out non-applicable):

  • Changes documented in CHANGES.md
  • Related issue exists and is referred to in the PR description and CHANGES.md
  • Added docstrings and API docs for any new/modified user-facing classes and functions
  • Changes/features documented in docs/*
  • Unit-tests adapted/added for changes/features
  • Test coverage remains or increases (target 100%)

@forman forman self-assigned this May 28, 2024
@forman forman requested a review from pont-us May 28, 2024 14:03
Copy link
Member

@pont-us pont-us left a comment

Choose a reason for hiding this comment

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

Approved, with a query (which might necessitate a change, depending on the answer to it).

CHANGES.md Outdated
@@ -1,10 +1,13 @@
## Version 0.7.1 (in development)
## Version 0.7.1 (from 2024-05-28)
Copy link
Member

Choose a reason for hiding this comment

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

Does the change from "in development" to "from 2024-05-28" mean that this commit corresponds to the 0.7.1 release? If so, the version number in __init__.py should lose the .dev0 suffix.

Copy link
Member Author

Choose a reason for hiding this comment

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

No, you are right, it should remain in development. There is no 0.7.1 release yet, but I planned it today, but now we have #97.

# https://opensource.org/licenses/MIT.

__version__ = "0.8.0.dev0"
__version__ = "0.7.1.dev0"
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if the .dev0 should be here -- see comment on CHANGES.md above.

Copy link
Member Author

Choose a reason for hiding this comment

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

Its ok, see above.

@forman forman merged commit f2a8222 into main May 28, 2024
@forman forman deleted the forman-93-return_num_slices branch May 28, 2024 14:52
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.

Return number of slices processed

3 participants