Giters

martanne / vis

A vi-like editor based on Plan 9's structural regular expressions

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

delete stale branches (at least 7yo or >500cb ones?)

goyalyashpal opened this issue · comments

commented

I have a local copy of all of these branches in case we ever need them, but it doesn't really bother me to have them here as well. The 0.8 branch I'd keep anyway, should we ever want to patch something on the 0.8 release.

Is there a downside on having stale branches on github?

commented

hi!

» 've a local copy of these branches
awesome

» 0.8 branch i'd keep
yeah, my main concern was for those branches being more than 7 years old; as they only have just 0,1,1,2,4,5,9 commits in them. now, i haven't checked the contents of those commits, but, if single commit content are not ginormously large, then number of commits can easily be dealt with (i.e. important parts merged in, important ones discarded)

» Is there a downside on having stale branches on github?
umh, the only thing is that if someone forks the repo with all the branches, then these branches will get copied.

all in all, no, no downside. just that i was thinking of forking the repo for some reason (oh yeah, recalled it - to just try to build it in my way) when i noticed all these branches.

commented

feel free to close as not planned if so 😃

commented

I guess cleaning up a bit cannot hurt. I've taken a closer look at them:

  • text-dump contains a rather interesting debug patch (to export text history as dot to make a graph) so I'll keep that around
  • events contains code to add more events to the LUA API, might still be interesting to pick up
  • sregex I dropped, this was an attempt to integrate sregexes but that project hasn't seen updates in 8 years
  • doc I'm not even sure why this existed so removed
  • selections was empty so removed
  • build fixes things on build which were reimplemented by now
  • configure was an outdated change in build as well
  • coverity_scan was more outdated build changes
  • tis same
  • scintillua I guess could still contain useful stuff until we're up to date with scintillua
commented

awesome 😊😇🚀

commented
  • was an attempt to integrate sregexes but that project hasn't seen updates in 8 years

how about re2 which is also DFA based

sregexes
A non-backtracking NFA/DFA-based Perl-compatible regex engine matching on large data streams

re2 https://github.com/google/re2
RE2 is a fast, safe, thread-friendly alternative to backtracking regular expression engines like those used in PCRE, Perl, and Python.

i wont think that big tech would mess up a regex engine as well... but these big techs cant be trusted either lol