Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • P PESummary
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 34
    • Issues 34
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 18
    • Merge requests 18
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • lscsoft
  • PESummary
  • Merge requests
  • !719

Enable a lot more flake8 warnings, but don't fail the CI

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Duncan Macleod requested to merge duncanmmacleod/pesummary:flake8-config into master Oct 03, 2022
  • Overview 1
  • Commits 1
  • Pipelines 2
  • Changes 3

This MR modifies the flake8 configuration in setup.cfg to display a lot more stuff, but don't have the CI job actually fail because of it, just present the issues in the merge request UI.

In this model the style job will only fail if flake8 crashes, which is worth looking into.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: flake8-config