Skip to content

feat: add branch name validation on renaming - #2081

Merged
extrawurst merged 4 commits into
gitui-org:masterfrom
sainad2222:master
Feb 20, 2024
Merged

extrawurst merged 4 commits into
gitui-org:masterfrom
sainad2222:master

Conversation

@sainad2222

@sainad2222 sainad2222 commented Feb 20, 2024

Copy link
Copy Markdown
Contributor

This Pull Request fixes/closes #2062 .

It changes the following:

  • Add branch name validation on renaming

I followed the checklist:

  • I added unittests
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog

@sainad2222 sainad2222 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason, test_pre_commit_py is panicking while running make check. I'll debug more and try to fix it

Edit: Issue is because in mac python is not available as /usr/bin/env. We need to be specific about python3. In linux also it shouldn't break by changing it. But let me know if something is breaking because of this

Comment thread CHANGELOG.md Outdated
sainad2222 and others added 3 commits February 21, 2024 00:57
lets not do this as on CI everything works fine. never change a running system
@extrawurst
extrawurst merged commit fa051ef into gitui-org:master Feb 20, 2024
IndianBoy42 pushed a commit to IndianBoy42/gitui that referenced this pull request Jun 4, 2024
Closes gitui-org#2062 

---------

Co-authored-by: extrawurst <776816+extrawurst@users.noreply.github.com>
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.

validate branch re-name

2 participants