fluid-project / fluid-lint-all

Consolidated linting logic free from any particular build technology
BSD 3-Clause "New" or "Revised" License
0 stars 5 forks source link

Update CI configuration. #67

Closed duhrer closed 2 months ago

duhrer commented 2 months ago

As @greatislander pointed out in the chats, our CI configuration uses a now-unsupported version of node in its tests.

There were also some test failures following the merger of #63, which were not observed in the code review. We should keep an eye out for errors like those observed here as part of the review.

In addition, it would be good to add a git configuration step before the tests, which now complain because of the lack of a default branch name:

hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint: 
hint:   git config --global init.defaultBranch <name>
hint: 
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint: 
hint:   git branch -m <name>
15:36:09.942:  Created git test fixture at '/tmp/fluid-lint-all-git-tests-172530'
error: unknown option `cached'