Skip to content
Snippets Groups Projects

Fix spelling mistakes

Merged Imported Fix spelling mistakes
codespell into master
Merged Imported Administrator requested to merge codespell into master

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Administrator
    Administrator @root · Imported
    Author Owner

    @z3ntu: thank you very much!

    Codespell seems to be very useful. Is there any reason why we should not integrate it into CI?

    By Oliver Smith on 2019-06-26T21:34:25

  • Administrator mentioned in commit 650ed492 · Imported

    mentioned in commit 650ed492

    By Luca Weiss on 2019-06-26T21:34:53

  • Administrator added 1 commit · Imported

    added 1 commit

    Compare with previous version

    By Oliver Smith on 2019-06-26T21:34:54

  • Administrator approved this merge request · Imported

    approved this merge request

    By Oliver Smith on 2019-06-26T21:35:43

  • Administrator enabled an automatic merge when the pipeline for 650ed492 succeeds · Imported

    enabled an automatic merge when the pipeline for 650ed492 succeeds

    By Oliver Smith on 2019-06-26T21:35:53

  • Administrator merged · Imported

    merged

    By Oliver Smith on 2019-06-26T21:49:57

  • Administrator
    Administrator @root · Imported
    Author Owner

    If we were to integrate it into the CI, we'd need to make a config for the excludes and where the spelling is intentional - because the tool does have false positives (e.g. it wants to change msdos to ms-dos all the time). And I don't think it's really worth it. Rather run it every month or so manually :)

    By Luca Weiss on 2019-06-27T11:29:49

Please register or sign in to reply