No description
Find a file
Vitalii Elenhaupt 320ec2ba75
Merge pull request #215 from kachick/follow-crystal-1.0.0
Follow crystal 1.0.0 in shard.yml
2021-03-23 18:20:46 +02:00
.github Remove ill-working CI hack 2021-01-16 19:57:57 +01:00
bench Exclude globs as arguments 2019-01-13 10:48:46 +02:00
bin Let it be extensible (#112) 2019-07-18 13:53:04 +03:00
spec Default to exclude_multiple_line_blocks in Style/VerboseBlock rule 2021-03-03 00:31:53 +01:00
src Merge branch 'master' into develop 2021-03-08 11:09:32 +02:00
.dockerignore Use make to build ameba in docker, add installation info 2018-08-25 13:55:07 +03:00
.editorconfig Hello, Ameba 2017-10-26 19:46:58 +03:00
.gitignore Switch CI from Travis to GitHub Actions 2021-01-12 02:56:47 +01:00
Dockerfile Use alpine:latest image in Dockerfile 2021-03-08 11:08:31 +02:00
LICENSE Update license years 2020-03-29 20:38:57 +03:00
Makefile Prevent error when run file exists 2019-11-29 21:55:24 +02:00
README.md Add support for showing end location marker (#200) 2021-01-26 08:38:19 +02:00
shard.yml Follow crystal 1.0.0 in shard.yml 2021-03-24 01:10:16 +09:00

Ameba

Code style linter for Crystal

(a single-celled animal that catches food and moves about by extending fingerlike projections of protoplasm)

About

Ameba is a static code analysis tool for the Crystal language. It enforces a consistent Crystal code style, also catches code smells and wrong code constructions.

See also Roadmap.

Usage

Run ameba binary within your project directory to catch code issues:

$ ameba
Inspecting 107 files

...............F.....................F....................................................................

src/ameba/formatter/flycheck_formatter.cr:4:33
[W] Lint/UnusedArgument: Unused argument `location`
> source.issues.each do |e, location|
                            ^

src/ameba/formatter/base_formatter.cr:12:7
[W] Lint/UselessAssign: Useless assignment to variable `s`
> return s += issues.size
         ^

Finished in 542.64 milliseconds
129 inspected, 2 failures

Run in parallel

Starting from 0.31.0 Crystal supports parallelism. It allows to run linting in parallel too. In order to take advantage of this feature you need to build ameba with preview_mt support:

$ crystal build src/cli.cr -Dpreview_mt -o bin/ameba
$ make install

Some quick benchmark results measured while running Ameba on Crystal repo:

$ CRYSTAL_WORKERS=1 ameba #=> 29.11 seconds
$ CRYSTAL_WORKERS=2 ameba #=> 19.49 seconds
$ CRYSTAL_WORKERS=4 ameba #=> 13.48 seconds
$ CRYSTAL_WORKERS=8 ameba #=> 10.14 seconds

Installation

As a project dependency:

Add this to your application's shard.yml:

development_dependencies:
  ameba:
    github: crystal-ameba/ameba
    version: ~> 0.13.0

Build bin/ameba binary within your project directory while running shards install.

You may also want to use it on Travis:

# .travis.yml
language: crystal
install:
  - shards install
script:
  - crystal spec
  - crystal bin/ameba.cr

Using this config Ameba will inspect files just after the specs run. Travis will also fail the build if some problems detected.

OS X

$ brew tap veelenga/tap
$ brew install ameba

Docker

Build the image:

$ docker build -t crystal-ameba/ameba .

To use the resulting image on a local source folder, mount the current (or target) directory into /src:

$ docker run -v $(pwd):/src crystal-ameba/ameba

Also available on DockerHub: https://hub.docker.com/r/veelenga/ameba

From sources

$ git clone https://github.com/crystal-ameba/ameba && cd ameba
$ make install

Configuration

Default configuration file is .ameba.yml. It allows to configure rule properties, disable specific rules and exclude sources from the rules.

Generate new file by running ameba --gen-config.

Sources

List of sources to run Ameba on can be configured globally via:

  • Globs section - an array of wildcards (or paths) to include to the inspection. Defaults to %w(**/*.cr !lib), meaning it includes all project files with *.cr extension except those which exist in lib folder.
  • Excluded section - an array of wildcards (or paths) to exclude from the source list defined by Globs. Defaults to an empty array.

In this example we define default globs and exclude src/compiler folder:

Globs:
  - **/*.cr
  - !lib

Excluded:
  - src/compiler

Specific sources can be excluded at rule level:

Style/RedundantBegin:
  Excluded:
  - src/server/processor.cr
  - src/server/api.cr

Rules

One or more rules, or a one or more group of rules can be included or excluded via command line arguments:

$ ameba --only   Lint/Syntax # runs only Lint/Syntax rule
$ ameba --only   Style,Lint  # runs only rules from Style and Lint groups
$ ameba --except Lint/Syntax # runs all rules except Lint/Syntax
$ ameba --except Style,Lint  # runs all rules except rules in Style and Lint groups

Or through the configuration file:

Style/RedundantBegin:
  Enabled: false

Explain issues

Ameba allows you to dig deeper into an issue, by showing you details about the issue and the reasoning by it being reported.

To be convenient, you can just copy-paste the PATH:line:column string from the report and paste behind the ameba command to check it out.

$ ameba crystal/command/format.cr:26:83           # show explanation for the issue
$ ameba --explain crystal/command/format.cr:26:83 # same thing

Inline disabling

One or more rules or one or more group of rules can be disabled using inline directives:

# ameba:disable Style/LargeNumbers
time = Time.epoch(1483859302)

time = Time.epoch(1483859302) # ameba:disable Style/LargeNumbers, Lint/UselessAssign

time = Time.epoch(1483859302) # ameba:disable Style, Lint

Editors & integrations

Credits & inspirations

Contributors

  • veelenga Vitalii Elenhaupt - creator, maintainer