mirror of
				https://source.denx.de/u-boot/u-boot.git
				synced 2025-11-04 02:11:25 +01:00 
			
		
		
		
	CI: Drop some jobs we didn't really utilize
- We have added more TODO/etc comments since this task was created and never focused on removing them. - The output of sloccount isn't preserved or looked at, and if desired should be in the release stats pages instead somehow. - The results of cppcheck aren't investigated and require modeling work to be useful to start with. Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
		
							parent
							
								
									562ed115e7
								
							
						
					
					
						commit
						b87f904709
					
				@ -67,16 +67,6 @@ stages:
 | 
				
			|||||||
                  :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
 | 
					                  :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
 | 
				
			||||||
                  :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
 | 
					                  :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  - job: cppcheck
 | 
					 | 
				
			||||||
    displayName: 'Static code analysis with cppcheck'
 | 
					 | 
				
			||||||
    pool:
 | 
					 | 
				
			||||||
      vmImage: $(ubuntu_vm)
 | 
					 | 
				
			||||||
    container:
 | 
					 | 
				
			||||||
      image: $(ci_runner_image)
 | 
					 | 
				
			||||||
      options: $(container_option)
 | 
					 | 
				
			||||||
    steps:
 | 
					 | 
				
			||||||
      - script: cppcheck -j$(nproc) --force --quiet --inline-suppr .
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  - job: docs
 | 
					  - job: docs
 | 
				
			||||||
    displayName: 'Build documentation'
 | 
					    displayName: 'Build documentation'
 | 
				
			||||||
    pool:
 | 
					    pool:
 | 
				
			||||||
@ -92,28 +82,6 @@ stages:
 | 
				
			|||||||
          make htmldocs KDOC_WERROR=1
 | 
					          make htmldocs KDOC_WERROR=1
 | 
				
			||||||
          make infodocs
 | 
					          make infodocs
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  - job: todo
 | 
					 | 
				
			||||||
    displayName: 'Search for TODO within source tree'
 | 
					 | 
				
			||||||
    pool:
 | 
					 | 
				
			||||||
      vmImage: $(ubuntu_vm)
 | 
					 | 
				
			||||||
    container:
 | 
					 | 
				
			||||||
      image: $(ci_runner_image)
 | 
					 | 
				
			||||||
      options: $(container_option)
 | 
					 | 
				
			||||||
    steps:
 | 
					 | 
				
			||||||
      - script: grep -r TODO .
 | 
					 | 
				
			||||||
      - script: grep -r FIXME .
 | 
					 | 
				
			||||||
      - script: grep -r HACK . | grep -v HACKKIT
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  - job: sloccount
 | 
					 | 
				
			||||||
    displayName: 'Some statistics about the code base'
 | 
					 | 
				
			||||||
    pool:
 | 
					 | 
				
			||||||
      vmImage: $(ubuntu_vm)
 | 
					 | 
				
			||||||
    container:
 | 
					 | 
				
			||||||
      image: $(ci_runner_image)
 | 
					 | 
				
			||||||
      options: $(container_option)
 | 
					 | 
				
			||||||
    steps:
 | 
					 | 
				
			||||||
      - script: sloccount .
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  - job: maintainers
 | 
					  - job: maintainers
 | 
				
			||||||
    displayName: 'Ensure all configs have MAINTAINERS entries'
 | 
					    displayName: 'Ensure all configs have MAINTAINERS entries'
 | 
				
			||||||
    pool:
 | 
					    pool:
 | 
				
			||||||
 | 
				
			|||||||
@ -151,22 +151,6 @@ check for new CONFIG symbols outside Kconfig:
 | 
				
			|||||||
        :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
 | 
					        :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
 | 
				
			||||||
        :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
 | 
					        :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
 | 
				
			||||||
 | 
					
 | 
				
			||||||
# QA jobs for code analytics
 | 
					 | 
				
			||||||
# static code analysis with cppcheck (we can add --enable=all later)
 | 
					 | 
				
			||||||
cppcheck:
 | 
					 | 
				
			||||||
  stage: testsuites
 | 
					 | 
				
			||||||
  script:
 | 
					 | 
				
			||||||
    - cppcheck -j$(nproc) --force --quiet --inline-suppr .
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
# search for TODO within source tree
 | 
					 | 
				
			||||||
grep TODO/FIXME/HACK:
 | 
					 | 
				
			||||||
  stage: testsuites
 | 
					 | 
				
			||||||
  script:
 | 
					 | 
				
			||||||
    - grep -r TODO .
 | 
					 | 
				
			||||||
    - grep -r FIXME .
 | 
					 | 
				
			||||||
    # search for HACK within source tree and ignore HACKKIT board
 | 
					 | 
				
			||||||
    - grep -r HACK . | grep -v HACKKIT
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
# build documentation
 | 
					# build documentation
 | 
				
			||||||
docs:
 | 
					docs:
 | 
				
			||||||
  stage: testsuites
 | 
					  stage: testsuites
 | 
				
			||||||
@ -177,12 +161,6 @@ docs:
 | 
				
			|||||||
    - make htmldocs KDOC_WERROR=1
 | 
					    - make htmldocs KDOC_WERROR=1
 | 
				
			||||||
    - make infodocs
 | 
					    - make infodocs
 | 
				
			||||||
 | 
					
 | 
				
			||||||
# some statistics about the code base
 | 
					 | 
				
			||||||
sloccount:
 | 
					 | 
				
			||||||
  stage: testsuites
 | 
					 | 
				
			||||||
  script:
 | 
					 | 
				
			||||||
    - sloccount .
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
# ensure all configs have MAINTAINERS entries
 | 
					# ensure all configs have MAINTAINERS entries
 | 
				
			||||||
Check for configs without MAINTAINERS entry:
 | 
					Check for configs without MAINTAINERS entry:
 | 
				
			||||||
  stage: testsuites
 | 
					  stage: testsuites
 | 
				
			||||||
 | 
				
			|||||||
@ -43,7 +43,6 @@ RUN apt-get update && apt-get install -y \
 | 
				
			|||||||
	clang-16 \
 | 
						clang-16 \
 | 
				
			||||||
	coreutils \
 | 
						coreutils \
 | 
				
			||||||
	cpio \
 | 
						cpio \
 | 
				
			||||||
	cppcheck \
 | 
					 | 
				
			||||||
	curl \
 | 
						curl \
 | 
				
			||||||
	device-tree-compiler \
 | 
						device-tree-compiler \
 | 
				
			||||||
	dosfstools \
 | 
						dosfstools \
 | 
				
			||||||
@ -105,7 +104,6 @@ RUN apt-get update && apt-get install -y \
 | 
				
			|||||||
	python3-virtualenv \
 | 
						python3-virtualenv \
 | 
				
			||||||
	rpm2cpio \
 | 
						rpm2cpio \
 | 
				
			||||||
	sbsigntool \
 | 
						sbsigntool \
 | 
				
			||||||
	sloccount \
 | 
					 | 
				
			||||||
	socat \
 | 
						socat \
 | 
				
			||||||
	softhsm2 \
 | 
						softhsm2 \
 | 
				
			||||||
	sparse \
 | 
						sparse \
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user