Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Still cannot run swiftlint on circleci #593

Closed
brennantaylor opened this issue Mar 24, 2016 · 9 comments
Closed

Still cannot run swiftlint on circleci #593

brennantaylor opened this issue Mar 24, 2016 · 9 comments
Labels
bug Unexpected and reproducible misbehavior.

Comments

@brennantaylor
Copy link

Still getting dyld: Library not loaded

I saw an issue was closed earlier but it doesn't look resolved.

In another project, swiftgen, it looks like a workaround was done in homebrew by not pouring from bottle if there is no /Applications/Xcode.app. Homebrew/legacy-homebrew#45008

This would at least get it running for now.

@norio-nomura
Copy link
Collaborator

Thanks for reporting.
We have PR #555 that will resolve this issue.

@brennantaylor
Copy link
Author

Oh I'm sorry I should have looked through the open pull requests more closely.

Thanks for the update!

@jpsim
Copy link
Collaborator

jpsim commented Apr 6, 2016

@brennantaylor how's this looking now that #555's been merged?

@jpsim jpsim added the bug Unexpected and reproducible misbehavior. label Apr 6, 2016
@brennantaylor
Copy link
Author

@jpsim I can't seem to compile HEAD with brew at the moment. Since there is no release either I can't easily install on circle. I can test as soon as the next release comes out.

brew install --HEAD swiftlint says ** CLEAN SUCCEEDED ** but then exits uncleanly.

@norio-nomura
Copy link
Collaborator

@brennantaylor That issue has been reproduced on my local mac.
It seems that is same as Carthage/Carthage#1243

@norio-nomura
Copy link
Collaborator

SwiftLint 0.10.0 has been released and is expected to run on CircleCI with brew install swiftlint.

@jpsim
Copy link
Collaborator

jpsim commented Apr 28, 2016

If someone confirms that it works, we'll close this issue, so just let us know!

@mikemee
Copy link

mikemee commented May 27, 2016

This worked on CircleCI just now:

  • SwiftLint installed ok with brew install swiftlint (see below)
  • the previous warning about SwiftLint not being installed went away
  • the build succeeded
  • an intentional fatal SwiftLint error in my code made the build fail
  • warnings did not fail (nor did they display anything in the logs -- I'll be making more warnings fail soon!)

Fwiw, my circle.yml file was (Carthage references left there to avoid unintentionally editing away some crucial detail, but see here if you're interested):

general:
  branches:
    only:
      - master

machine:
  xcode:
    version: 7.3.1

dependencies:
  override:
    - brew install swiftlint
    - bin/bootstrap-if-needed
  cache_directories:
    - "Carthage"

Awesome awesome tool. Thanks Realm for making it available and supporting it. (And, together with the video transcriptions, this made me find realm and now I'm using it my apps...)

@jpsim
Copy link
Collaborator

jpsim commented May 27, 2016

Great to hear this works now 😄

@jpsim jpsim closed this as completed May 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected and reproducible misbehavior.
Projects
None yet
Development

No branches or pull requests

4 participants