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

feat: add branch name validation on renaming #2081

Merged
merged 4 commits into from
Feb 20, 2024

Conversation

sainad2222
Copy link
Contributor

@sainad2222 sainad2222 commented Feb 20, 2024

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

Copy link
Contributor Author

@sainad2222 sainad2222 left a comment

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

CHANGELOG.md Outdated Show resolved Hide resolved
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 extrawurst:master Feb 20, 2024
18 checks passed
IndianBoy42 pushed a commit to IndianBoy42/gitui that referenced this pull request Jun 4, 2024
Closes extrawurst#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