Skip to content

on-demand branchname validity (while typing) #559

Closed

Description

@extrawurst

libgit2 exposes a helper function to check a branch name validity: https://libgit2.org/libgit2/#HEAD/group/branch/git_branch_name_is_valid

lets run this against the input before it gets confirmed.
this way we can give live feedback to the user whether the name is valid.

Activity

  1. franciscod commented on May 29, 2021

    @franciscod

    hello there! rust newcomer here. will take a shot at this :)

  2. extrawurst commented on May 29, 2021

    @extrawurst
    CollaboratorAuthor

    hello there! rust newcomer here. will take a shot at this :)

    Very cool, welcome! Let me know when you need any help

  3. extrawurst commented on May 29, 2021

    @extrawurst
    CollaboratorAuthor

    requires upstream change first: rust-lang/git2-rs#714

  4. franciscod commented on May 29, 2021

    @franciscod
  5. changed the title [-]on-demand branchname validity [/-] [+]on-demand branchname validity (while typing)[/+] on May 29, 2021
  6. franciscod commented on May 29, 2021

    @franciscod

    also wip: master...franciscod:wip_check_branch_name

    (won't create PR until the one in git2-rs lands)

  7. extrawurst commented on Jun 1, 2021

    @extrawurst
    CollaboratorAuthor

    @franciscod congrats for the merge in git2-rs but it did not contain the same function for tags which we also wanted to cover in here :(

  8. franciscod commented on Jun 1, 2021

    @franciscod

    Thanks! git_branch_name_is_valid is the only one that I implemented. Internally it also checks that it's a valid name for a reference. Do we need anything else?

  9. franciscod commented on Jun 1, 2021

    @franciscod

    ah, for tags, missed that one! Well #759 is my work in progress for branches, I guess we could do branches first and then tags?

  10. extrawurst commented on Aug 10, 2021

    @extrawurst
    CollaboratorAuthor

    after rust-lang/git2-rs#734 we can at least go ahead with the branch validity check

  11. added this to the v0.17 milestone on Aug 15, 2021
  12. added a commit that references this issue on Aug 15, 2021
    d42b003
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

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions