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

Improved control over code changes #925

Merged
merged 22 commits into from
Aug 16, 2024
Merged

Improved control over code changes #925

merged 22 commits into from
Aug 16, 2024

Conversation

jesper-friis
Copy link
Collaborator

@jesper-friis jesper-friis commented Aug 14, 2024

Description

Implemented the formalised code changes described in PR #920.

Benefits: Make it possible to improve our control over code changes. Provide predictability for when changes will happen and give users a time-span to a given versions number to adapt to new changes.

Motivation: This PR was motivated by PR #907. When merged, we can address PR #907 as our first behavior change.


Possible further work: Automatic generation of a page in the online documentation that lists upcoming versions including (for each upcoming version):

  • deprecated features that are planned to be removed
  • new behavior will be enabled by default
  • old behavior that is planned to be removed

Would such a page be useful? It would not include planned new features (for that we can use github issues and milestones).

Type of change

  • Bug fix & code cleanup
  • New feature
  • Documentation update
  • Test update

Checklist for the reviewer

This checklist should be used as a help for the reviewer.

  • Is the change limited to one issue?
  • Does this PR close the issue?
  • Is the code easy to read and understand?
  • Do all new feature have an accompanying new test?
  • Has the documentation been updated as necessary?

@jesper-friis jesper-friis changed the title Behavior Improved control over code changes Aug 15, 2024
jesper-friis and others added 5 commits August 16, 2024 16:17
Co-authored-by: Francesca L. Bleken <48128015+francescalb@users.noreply.github.com>
Co-authored-by: Francesca L. Bleken <48128015+francescalb@users.noreply.github.com>
@jesper-friis jesper-friis merged commit bd8dc6b into master Aug 16, 2024
14 checks passed
@jesper-friis jesper-friis deleted the behavior branch August 16, 2024 15:26
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.

2 participants