Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • pleroma pleroma
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 647
    • Issues 647
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 88
    • Merge requests 88
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PleromaPleroma
  • pleromapleroma
  • Merge requests
  • !488

#292 Replace try/rescue with File.Exists? in dev.exs

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged rinpatch requested to merge rinpatch/pleroma:fix/config-replace-try-catch-with-file-exists into develop Dec 01, 2018
  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 1

The problem with try/rescue is that incorrect dev.secrets.exs will result in a warning that it does not exist. With this patch it will result in compilation error, thus being immediately noticed. The solution was taken from here

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/config-replace-try-catch-with-file-exists