Skip to content

CI/[misc_checks]: run [lint.sh --check-licenses-git-new]

Arvid Jakobsson requested to merge arvid@check-license--activate into master

Context

I had missed to add the --check-licenses-git-new check in the CI in the original MR !10331 (merged) :/

Here, I make sure it runs. I do not add it to the target check-linting since the action --check-licenses-git-new only makes sense when CHECK_LICENSES_DIFF_BASE is set. We know the value in the CI, but if developers run it locally they may end up with confusing results.

Manually testing the MR

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR

Merge request reports