Skip to content

Move the SwiftLint build phase to a separate dedicated target#206

Merged
mattrubin merged 1 commit into
developfrom
lint-target
Apr 27, 2019
Merged

Move the SwiftLint build phase to a separate dedicated target#206
mattrubin merged 1 commit into
developfrom
lint-target

Conversation

@mattrubin

Copy link
Copy Markdown
Owner

Lint warnings are important during development of the framework, but are usually irrelevant for users of the framework.

This change allows the OneTimePassword schemes to trigger the SwiftLint run when building the framework directly, but will allow schemes of projects embedding the framework to build it without seeing lint warnings.

This allows the OneTimePassword schemes to trigger the SwiftLint run when building the framework directly, but will allow other projects embedding the framework to build it without seeing lint warnings that are unlikely to be relevant to users of the framework.
@codecov

codecov Bot commented Apr 27, 2019

Copy link
Copy Markdown

Codecov Report

Merging #206 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #206   +/-   ##
========================================
  Coverage    99.34%   99.34%           
========================================
  Files            6        6           
  Lines          153      153           
========================================
  Hits           152      152           
  Misses           1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b8e8b7e...e2a9b6c. Read the comment docs.

@mattrubin mattrubin merged commit 2414c60 into develop Apr 27, 2019
@mattrubin mattrubin deleted the lint-target branch April 27, 2019 22:33
@mattrubin mattrubin added this to the 3.1.6 milestone Sep 18, 2019
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.

1 participant