Skip to content

Commit

Permalink
[nrf noup] ci: NCS-specific CI tweaks
Browse files Browse the repository at this point in the history
Necessary changes for NCS CI.

- compliance: Add an additional check for sauce tags.
  Also, since, in the code, we refer to Kconfig symbols that are defined
  in the sdk-nrf repository, the Kconfig check will not pass. Use
  KconfigBasic instead, which will not grep the code for instances
  of CONFIG_.
- Add a Jenkinsfile.
- Extend the max commit line lengths for Gitlint.

Signed-off-by: Carles Cufi <[email protected]>
(cherry picked from commit 83f7974)
  • Loading branch information
carlescufi authored and mbolivar-nordic committed Feb 2, 2022
1 parent 45ef0d2 commit 05bde23
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/compliance.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ jobs:
git config --global user.email "[email protected]"
git config --global user.name "Your Name"
git remote -v
# Sauce tag checks before rebasing
git log --oneline --first-parent origin/${BASE_REF}..HEAD | grep -E -v "\[nrf (mergeup|fromtree|fromlist|noup)\]" && { echo 'Sauce tag missing'; exit 1; }
git rebase origin/${BASE_REF}
# debug
git log --pretty=oneline | head -n 10
Expand All @@ -69,7 +71,7 @@ jobs:
# debug
ls -la
git log --pretty=oneline | head -n 10
./scripts/ci/check_compliance.py -m Codeowners -m Devicetree -m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m Kconfig -c origin/${BASE_REF}..
./scripts/ci/check_compliance.py -m Codeowners -m Devicetree -m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m KconfigBasic -c origin/${BASE_REF}..
- name: upload-results
uses: actions/upload-artifact@master
Expand Down
4 changes: 2 additions & 2 deletions .gitlint
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ debug = false
extra-path=scripts/gitlint

[title-max-length-no-revert]
line-length=75
line-length=120

[body-min-line-count]
min-line-count=1
Expand All @@ -39,7 +39,7 @@ words=wip

[max-line-length-with-exceptions]
# B1 = body-max-line-length
line-length=75
line-length=120

[body-min-length]
min-length=3
Expand Down
5 changes: 5 additions & 0 deletions Jenkinsfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
@Library("CI_LIB") _

def pipeline = new ncs.sdk_zephyr.Main()

pipeline.run(JOB_NAME)

0 comments on commit 05bde23

Please sign in to comment.