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

Fix make package command #3890

Merged
merged 1 commit into from
Mar 11, 2022
Merged

Fix make package command #3890

merged 1 commit into from
Mar 11, 2022

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Mar 10, 2022

Fixes #3815
Fixes #3887

@jpsim
Copy link
Collaborator Author

jpsim commented Mar 10, 2022

Verified installing the package in a fresh macOS 12.2.1 VM only installs a single swiftlint binary:

image

@jpsim jpsim marked this pull request as ready for review March 10, 2022 22:23
@jpsim
Copy link
Collaborator Author

jpsim commented Mar 10, 2022

Can you please take a look @miketimofeev & @jberkel?

--root "$(TEMPORARY_FOLDER)" \
--version "$(VERSION_STRING)" \
--root "$(PACKAGE_ROOT)" \
--version $(VERSION_STRING) \
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was previously being double-quoted

@codecov-commenter
Copy link

Codecov Report

Merging #3890 (7117ea2) into master (3273cef) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3890      +/-   ##
==========================================
- Coverage   91.99%   91.99%   -0.01%     
==========================================
  Files         435      435              
  Lines       22074    22074              
==========================================
- Hits        20307    20306       -1     
- Misses       1767     1768       +1     
Impacted Files Coverage Δ
...iftLintFramework/Extensions/String+SwiftLint.swift 89.39% <0.00%> (-1.52%) ⬇️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@jpsim jpsim merged commit 92e1996 into master Mar 11, 2022
@jpsim jpsim deleted the fix-make-package-command branch March 11, 2022 01:00
jpsim added a commit that referenced this pull request Mar 16, 2022
coffmark pushed a commit to coffmark/SwiftLint that referenced this pull request Apr 11, 2022
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.

swiftlint reports wrong version 0.46.1 & 0.46.2 package installation fails
2 participants