Skip to content

add a --configuration option to pod lint. - #9760

Merged
amorde merged 1 commit into
CocoaPods:masterfrom
gereons:lint-configuration
May 7, 2020
Merged

amorde merged 1 commit into
CocoaPods:masterfrom
gereons:lint-configuration

Conversation

@gereons

@gereons gereons commented May 2, 2020

Copy link
Copy Markdown
Contributor

This implements #9686

@gereons

gereons commented May 3, 2020

Copy link
Copy Markdown
Contributor Author

Tbh I have no clue what's causing the test failure on CircleCI. Are there any actionable errors in the log besides "CompileSwift failed with a nonzero exit code"?

@paulb777

paulb777 commented May 4, 2020

Copy link
Copy Markdown
Member

The CircleCI issue might be related to #9761?

Is it possible to add a test to spec/functional/command/repo/push_spec.rb?

@gereons

gereons commented May 4, 2020 •

Copy link
Copy Markdown
Contributor Author

If I understand #9761 correctly, it just removes the creation of a file that is ignored anyway, so I don't think it's related.

Also, I'm not sure if .../repo/push_spec.rb is the right place for testing this? Looking at those files, ../lib/list_spec.rc seems to be more suitable to me. I can certainly add a test there.

Comment thread lib/cocoapods/command/lib/lint.rb
@gereons
gereons force-pushed the lint-configuration branch from ca8be2d to 7d68f49 Compare May 7, 2020 08:00
@amorde amorde added this to the 1.10.0 milestone May 7, 2020
@amorde
amorde merged commit 0121759 into CocoaPods:master May 7, 2020
@amorde

amorde commented May 7, 2020

Copy link
Copy Markdown
Member

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants