Skip to content

Conversation

@jackVanish
Copy link
Contributor

Adds the check and read-in for parquet files to the read_otn_detections process.

@mhpob
Copy link
Collaborator

mhpob commented Sep 10, 2025

I think the double PR was made as this is requesting merge into main whereas the other was requesting merge into dev.

I think main is closed now, right? So this should go to dev

@jackVanish jackVanish changed the base branch from main to dev September 10, 2025 16:24
@jackVanish jackVanish requested a review from jdpye as a code owner September 10, 2025 16:24
@jackVanish
Copy link
Contributor Author

Sure, it can go into dev. I thought main was appropriate since this was going to be part of the next release but if that's not the flow then that's no problem.

@mhpob
Copy link
Collaborator

mhpob commented Sep 10, 2025

Oh, I may be the one misunderstanding. @chrisholbrook which branch do you want PRs to go to?

@chrisholbrook
Copy link
Collaborator

Please target dev with PRs containing new features, to be included in next release. When dev first deviates from main, advance version number (e.g., in this case dev should become v0.9.0.9001) and add new entry in NEWS.
Bug fixes can target main. In that case, they are merged with main, then main-->dev.

@jackVanish
Copy link
Contributor Author

Right, PR targets dev now.,

@chrisholbrook chrisholbrook merged commit e9ce0fa into dev Sep 12, 2025
7 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.

4 participants