Skip to content

Conversation

@zackgalbreath
Copy link
Contributor

No description provided.

@williamjallen
Copy link
Collaborator

Should this be merged into 3.3, given that we already have a few other things to get into that release?

@zackgalbreath
Copy link
Contributor Author

Should this be merged into 3.3, given that we already have a few other things to get into that release?

Yes. Merge to master first then backport, or vice versa?

@williamjallen
Copy link
Collaborator

Yes. Merge to master first then backport, or vice versa?

If we merge this into 3.3, we can then merge 3.3 into master quickly thereafter. The only time merging changes into the release branches is problematic is when those changes linger for months on end like what happened with the 3.2 release process. I've been trying to merge 3.3 into master every time I make a new release candidate, and I try to make a new release candidate at least weekly, which keeps master relatively up-to-date.

Replace "build name contains" with "build name is"
to more accurately reflect the functionality that was changed in
It was confusing to display this caption below the table it references.
@zackgalbreath zackgalbreath force-pushed the update_dynamic_rules_label branch from ea5c2f5 to 3f8a3a9 Compare February 21, 2024 19:30
@zackgalbreath zackgalbreath changed the base branch from master to releases/3.3 February 21, 2024 19:30
@williamjallen williamjallen added this pull request to the merge queue Feb 21, 2024
Merged via the queue into releases/3.3 with commit fff13e6 Feb 21, 2024
@williamjallen williamjallen deleted the update_dynamic_rules_label branch February 21, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants