FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[maintenance] Branche maintenance · Issue #6782 · Codeception/Codeception · GitHub

Repository navigation

[maintenance] Branche maintenance #6782

Closed

Description

I propose removing all old branches to simplify maintenance and clear things up for (potential) contributors.

  1. Keeping old branches like 3.1 has no purpose, we can always go back to 3.1 from the current branch.
  2. We don't actively support multiple branches, and if we do, we shouldn't there's no bandwidth for it.

I propose:

  • Renaming the primary branch to main
  • Dropping all version branches
  • Setting up automated releases on the master branch using semantic versioning and semantic commit messages

@Naktibalda @DavertMik @TavoNiievez

I can do all of these, just want some people to agree before I make changes like this.

Activity

  1. Naktibalda commented on Sep 11, 2024

    Member

    master branch should be created from 5.2.
    Please keep 5.1 if you think that 5.1 is still supported.

    In the days of Codeception 2 and 3, when modules were part of Codeception/Codeception repository, documentation pages contained links to source code is particular branches.
    There was even a row of versions to choose from, but these links had been removed recently (in the last 2 years or so).

  2. TavoNiievez commented on Sep 12, 2024

    Member

    Ok with the changes.
    Just main instead of master.

  3. SamMousa commented on Sep 12, 2024

    CollaboratorAuthor

    Ok with the changes. Just main instead of master.

    Out of respect for your work im going to agree with it just this once, even though i very strongly disagree with this! Im happy to have a discussion about it, but this is probably not the place for political/ideological discussions ;-)
    If the others agree with it too ofc

  4. TavoNiievez commented on Sep 12, 2024

    Member

    very thoughtful of you, thank you.
    Let's continue.

  5. SamMousa commented on Sep 12, 2024

    CollaboratorAuthor

    Someone with permissions needs to these:

    • Remove all branch protection
    • Make main branch protected
  6. SamMousa commented on Sep 12, 2024

    CollaboratorAuthor

    Please keep 5.1 if you think that 5.1 is still supported.

    @Naktibalda I propose removing it, we can always recreate it from a tag. We don't really want to support 5.1 since 5.2.x should be backwards compatibility anyway.

  7. Naktibalda commented on Sep 12, 2024

    Member

    @SamMousa I granted you and @TavoNiievez some extra permissions.

  8. DavertMik commented on Sep 15, 2024

    Member

    I think version-named branches are fine, maybe they could be improved as 5.x or 6.x
    This was a good decision as

    • you don't need to bother of main vs master
    • you can easily maintain backports for previous versions if needed

    But I agree with the decision you've made. Thank you all for your involvement!

  9. SamMousa commented on Sep 15, 2024

    CollaboratorAuthor

    We dont have bandwidth for regular maintenance let alone backports xD the nice thing about 1 branch is automated releases based on commit messages, thats the dream when it comes to low cost maintenance.

    Also it makes contribution easier and more worthwhile. I see a bug, i fix it and if it gets merged i can instantly use it. Instead of putting your own fork in composer and forgetting about it for months etc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL