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

Fixes for group_index_step validator (#1143) #1152

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

lucas-flexcompute
Copy link
Collaborator

@momchil-flex I'm not sure this goes into 2.4, given the timing, so I added the changes in the "unreleased" section of the changelog.

@momchil-flex
Copy link
Collaborator

momchil-flex commented Sep 11, 2023

Let's get it in, but it needs to be merged in develop then. And move to 2.4.0 changelog.

@lucas-flexcompute lucas-flexcompute changed the base branch from pre/2.4 to develop September 11, 2023 15:37
@lucas-flexcompute
Copy link
Collaborator Author

Let's get it in, but it needs to be merged in develop then. And move to 2.4.0 changelog.

Rebased to develop, but there's no 2.4 section in the CHANGELOG. Do I create one?

@momchil-flex
Copy link
Collaborator

Oh man. My bad. I merged the docs 2.4 branch to develop but not this one, only the other way round, develop into pre/2.4. Sorry about that. We can do it either way now, I can merge 2.4 into develop and you can update the PR, or you can revert it to 2.4. Sorry again.

@lucas-flexcompute
Copy link
Collaborator Author

No problem, I'll revert to 2.4 then.

Signed-off-by: Lucas Heitzmann Gabrielli <lucas@flexcompute.com>
@lucas-flexcompute lucas-flexcompute changed the base branch from develop to pre/2.4 September 11, 2023 15:56
@momchil-flex momchil-flex merged commit f94efba into pre/2.4 Sep 11, 2023
16 checks passed
@momchil-flex momchil-flex deleted the lucas/group_index_validator_fixes branch September 11, 2023 17:41
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.

None yet

3 participants