Skip to content

Conversation

@kaitj
Copy link
Collaborator

@kaitj kaitj commented Jan 23, 2024

Proposed changes

This PR primarily updates dependencies (namely snakebids) to resolve some outstanding (non-workflow related) bugs. Also updates the version of the actions workflows being used.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionalitiy)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (if none of the other choices apply)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

  • Changes have been tested to ensure that fix is effective or that a feature works.
  • Changes pass the unit tests
  • Code has been run through the poe quality task
  • I have included necessary documentation or comments (as necessary)
  • Any dependent changes have been merged and published

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

Jason Kai added 4 commits January 23, 2024 07:45
Update snakebids to ^0.10.2, which fixes a dependency issue causing
failure in building documentation. Also used this commit as a way to
update all other dependencies.
Make use of ruff for formatting tasks (e.g. isort, black).
@kaitj kaitj added the maintenance Updates or improvements that do not change functionality of the code label Jan 23, 2024
@kaitj kaitj merged commit f765d86 into khanlab:main Jan 23, 2024
@kaitj kaitj deleted the maint/dep-updates branch January 23, 2024 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Updates or improvements that do not change functionality of the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant