search

match4everyone / match4everything

Other
7 stars 0 forks source link

Tidy up scripts #182

Closed Baschdl closed 3 years ago

Baschdl commented 3 years ago

@bjrne @maltezacharias What do you think? Do you have an idea for a cleaner solution?

maltezacharias commented 3 years ago

Please review above suggestions. The PR looks fine to me with or without them.

bjrne commented 3 years ago

Do you have an idea for a cleaner solution?

I don't :sweat_smile:

Baschdl commented 3 years ago

Is the change in permissions intended?

Yes, it makes it executable

Baschdl commented 3 years ago

@bjrne I would wait for the answer from @maltezacharias but I'm also missing this script in the other branches

maltezacharias commented 3 years ago

@bjrne I would wait for the answer from @maltezacharias but I'm also missing this script in the other branches

please ping on telegram if the answer is not sufficient, don't want to keep you waiting