upstream/kallithea Files · scripts/whitespacecleanup.sh

validators: don't catch all Exceptions as invalid clone URIs, be specific...

validators: don't catch all Exceptions as invalid clone URIs, be specific When adding a new repository with a remote clone URI, the URI will be validated in some way. Several exceptions could occur during that validation. Previously, the code would catch based on 'Exception', which means that _any_ exception would cause the URI to be found invalid. This means that errors in the code (e.g. related to Python 3 conversion) were also categorized as 'invalid clone URI'. And thus, the tests that test an actually invalid URI would pass, even though there was a bug. Now, things have been refactored so it only is relevant to catch InvalidCloneUriException. Any other exception will now yield a 500 Internal Server Error, as expected.

Mads Kiilerich - - Load All Authors

File last commit:

r7782:0288dd05 default


                r8155:4aa6a17a

default

Download file

             whitespacecleanup.sh
        
                    23 lines
            
             | 1.0 KiB
            
                | application/x-sh
            
             |
                BashLexer
            
             / scripts / whitespacecleanup.sh
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      #!/bin/bash -x

      # Enforce some consistency in whitespace - just to avoid spurious whitespaces changes

      files=`hg mani | egrep -v '/fontello/|/email_templates/|(^LICENSE-MERGELY.html|^docs/Makefile|^scripts/whitespacecleanup.sh|/(graph|mergely|native.history)\.js|/test_dump_html_mails.ref.html|\.png|\.gif|\.ico|\.pot|\.po|\.mo|\.tar\.gz|\.diff)$'`

      sed -i "s/`printf '\r'`//g" $files

      sed -i -e "s,`printf '\t'`,    ,g" $files

      sed -i -e "s,  *$,,g" $files

      sed -i -e 's,\([^ ]\)\\$,\1 \\,g' -e 's,\(["'"'"']["'"'"']["'"'"']\) \\$,\1\\,g' $files

      # ensure one trailing newline - remove empty last line and make last line include trailing newline:

      sed -i -e '$,${/^$/d}' -e '$a\' $files

      sed -i -e 's,\([^ /]\){,\1 {,g' `hg loc '*.css'`

      sed -i -e 's|^\([^ /].*,\)\([^ ]\)|\1 \2|g' `hg loc '*.css'`

      hg mani | xargs chmod -x

      hg loc 'set:!binary()&grep("^#!")&!(**_tmpl.py)&!(**/template**)' | xargs chmod +x

      # isort is installed from dev_requirements.txt

      isort --line-width 160 --wrap-length 160 --lines-after-imports 2 `hg loc '*.py'`

      hg diff

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				#!/bin/bash -x

				# Enforce some consistency in whitespace - just to avoid spurious whitespaces changes

				files=`hg mani \| egrep -v '/fontello/\|/email_templates/\|(^LICENSE-MERGELY.html\|^docs/Makefile\|^scripts/whitespacecleanup.sh\|/(graph\|mergely\|native.history)\.js\|/test_dump_html_mails.ref.html\|\.png\|\.gif\|\.ico\|\.pot\|\.po\|\.mo\|\.tar\.gz\|\.diff)$'`

				sed -i "s/`printf '\r'`//g" $files
				sed -i -e "s,`printf '\t'`, ,g" $files
				sed -i -e "s, *$,,g" $files
				sed -i -e 's,\([^ ]\)\\$,\1 \\,g' -e 's,\(["'"'"']["'"'"']["'"'"']\) \\$,\1\\,g' $files
				# ensure one trailing newline - remove empty last line and make last line include trailing newline:
				sed -i -e '$,${/^$/d}' -e '$a\' $files

				sed -i -e 's,\([^ /]\){,\1 {,g' `hg loc '*.css'`
				sed -i -e 's\|^\([^ /].,\)\([^ ]\)\|\1 \2\|g' `hg loc '.css'`

				hg mani \| xargs chmod -x
				hg loc 'set:!binary()&grep("^#!")&!(_tmpl.py)&!(/template**)' \| xargs chmod +x

				# isort is installed from dev_requirements.txt
				isort --line-width 160 --wrap-length 160 --lines-after-imports 2 `hg loc '*.py'`

				hg diff