On Fri, Oct 18, 2013 at 04:56:56AM -0700, Marcin Ku??mi??ski wrote:
> > @@ -74,6 +74,28 @@ def update_server_info(repo="."):
> >      server_update_server_info(r)
> >  
> >  
> > +def set_symbolic_ref(repo=".", ref_name=None, force=False, outstream=sys.stdout,
> > +                     errstream=sys.stderr):
> > +    """
> > +    Set git symbolic ref into HEAD
> > +
> > +    :param repo: path to the repository
> > +    :param ref_name: short name of the new ref
> > +    :param force: force settings  without checking if it exists in refs/heads
> > +    :param outstream: Output stream (defaults to stdout)
> > +    :param errstream: Error stream (defaults to stderr)
> > +    """
> > +    if ref_name:
> 
> Should we show an error if ref_name is not given, removing the non-optional None would be odd for arguments, all later would have to become optional also.

I think it makes sense to have repo as a mandatory argument; that way ref_name can be mandatory as well.

Cheers,

Jelmer