Enforce coding conventions (#1452) · attackgithub/web-frameworks@585fd5f · GitHub
Skip to content

Commit 585fd5f

Browse files
authored
Enforce coding conventions (the-benchmarker#1452)
* lint ruby source with rubocop * lint python * activate php code sniffer * fix ruby style violations * configure jshint on hound * enable eslint * wrong magic comment * enable python linter (flake8) * format elixir source * explain what is E501 in python * lint nim / swift sources * remove <go get> from go frameworks containers * ci -> fix syntax error on lint script * ci -> install npm for clang checks * ci -> use dist binary to run clang linter * refactor(ruby): Use community guidelines -> rubocop * refactor(crystal): Use built-in tool to reformat * refactor(go): Use gofmt to format code * refactor(java): Use clang guide lines (google variant) to format code * refactor(javascript): Reformat source code with prettier * fix(ci): Fix nimpretty installation * feat(ci): Enable multi-stage CI * feat(ci): Use OSX worker for Objective C / Swift lint * feat(ci): Remove elixir test since disabled * fix(ci): Do not use sudo for composer * fix(ci): Do not use userland to install black * fix(ci): Add rubocop configuration * fix(ci): Install coursier for scala artifacts * fix(ci): Use clang-format to check C/C++/Objective C/Java * fix(ci): Avoid using unary operator in bash * test(ci): Test c lint * fix(ci): Fix PHP lint * feat(ci): Complete test suite * refactor(python): Reformat source with black * refactor(python): Prefer double quotes * feat(ci): Add an automatic tool to create ci config * feat(lint): Add rubocop configuration file * refactor(ruby): Lint files * chore(go): Remove sum files * chore(ci): Build CI config from built-in tool * feat(ci): Use OSX workers for swift * fix(ci): Use built-in to define lint policy * feat(ci): Be more verbose on lint stage * fix(ci): Use short params for clang-format * fix(ci): Be verbose on C++ lint * feat(ci): Add kotlin support * chore(ci): Show clang-format version * feat(ci): Use xenial to ensure more updated packages / tools * fix(ci): Force rustfmt installation * fix(ci): clang-format should modify actual files * fix(ci): Install swift lint for swift projects * feat(ci): Add support for kotlin * refactor(kotlin): Refactor source with ktlint * fix(ci): Update brew formulas to install swiftlint * fix(ci): CI was not running on node * fix(ci): typo * fix(ci): Remove unnecessary brew command * fix(ci): Use the last LTS version of nodejs * fix(ci): Use swift travis vim for swift instead of objective c * refactor(swift): Avoid using too short variable name * fix(ci): Use the latest version of swiftlint * fix(ci): Use travis swiftlint version * fix(ci): Lint each kotlin source instead of whol directory * refactor(kotlin): Disable ktlin wildcard import warning * fix(ci): Be less strict on swift lint * feat(ci): Be more verbose on php linting * fix(ci): Update package before installing nim * feat(ci): Use an external script on test * refactor(ci): Remove unused exit (lint script will exit immediately if error) * fix(ci): Wrong ktlin usage * fix(ci): Fix c lint since wrong argument used * feat(ci): Update to bionic for more recent packages * refactor(ci): Avoid specifying node version (use default) * refactor(ci): Prefer system packages * Use nodejs version 10 (because LTS) instead of 0.10 This reverts commit 7373ed6. * fix(ci): Always update apt before installing a package * feat(ci): Use an up-to-date nim version (with nimpretty) * refactor(javascript): Use standardjs coding guidelines * chore: Remove unused files * refactor(ruby): Disable a minimum of rubocop options * fix(ci): Do not lint json files with standardjs
1 parent 8cf0c5a commit 585fd5f

148 files changed

Lines changed: 1513 additions & 906 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.ci/lint.sh‎

Lines changed: 117 additions & 3 deletions

‎.ci/mapping.yml‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
python:
2+
language: python
3+
crystal:
4+
language: crystal
5+
php:
6+
language: php
7+
swift:
8+
os: osx
9+
language: swift
10+
csharp:
11+
language: csharp
12+
go:
13+
language: go
14+
cpp:
15+
language: cpp
16+
scala:
17+
language: scala
18+
node:
19+
language: node_js
20+
ruby:
21+
language: ruby
22+
rust:
23+
language: rust
24+
c:
25+
language: c
26+
nim:
27+
language: minimal
28+
java:
29+
language: java
30+
kotlin:
31+
language: java

‎.ci/template.mustache‎

Lines changed: 0 additions & 28 deletions
This file was deleted.

‎.ci/test.sh‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
#!/bin/bash
2+
3+
set -eu
4+
5+
shards install
6+
shards build
7+
bin/make config
8+
bin/make neph_config
9+
bin/neph ${FRAMEWORK} --mode=CI --seq
10+
crystal spec

‎.jshintrc‎

Lines changed: 0 additions & 3 deletions
This file was deleted.

‎.rubocop.yml‎

Lines changed: 16 additions & 0 deletions

0 commit comments

Comments
 (0)