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

Issue/654/redshift to utils #655

Merged
merged 12 commits into from
Feb 16, 2025
Merged

Issue/654/redshift to utils #655

merged 12 commits into from
Feb 16, 2025

Conversation

m-aguena
Copy link
Collaborator

@m-aguena m-aguena commented Dec 6, 2024

Closes #654

@coveralls
Copy link

coveralls commented Jan 23, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling d1a6147 on issue/654/redshift_to_utils
into 5b20c76 on main.

@m-aguena m-aguena marked this pull request as ready for review February 7, 2025 15:53
@combet combet self-requested a review February 13, 2025 15:03
Copy link
Collaborator

@combet combet left a comment

Choose a reason for hiding this comment

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

Thank you @m-aguena - this file reorganisation makes sense to me and makes the code base clearer. All the notebooks appear to be running fine and tests are happy :)

@hsinfan1996
Copy link
Collaborator

The version is 1.15.1 in the main branch.

@m-aguena
Copy link
Collaborator Author

The version is 1.15.1 in the main branch.

thanks for the head up

@m-aguena m-aguena force-pushed the issue/654/redshift_to_utils branch 2 times, most recently from 2585d96 to a31fe79 Compare February 16, 2025 01:01
@m-aguena m-aguena merged commit c72eadc into main Feb 16, 2025
2 checks passed
@hsinfan1996
Copy link
Collaborator

Sorry, I think the version number is not correct. Shouldn't it be 1.16.0?

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.

Move redshift module into utils
4 participants