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

Support SPM installation #1379

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

allenhumphreys
Copy link

Fixes #1372 #1370

To test:

You can add .package(url: "https://github.com/allenhumphreys/AztecEditor-iOS", branch: "ah/spm") to the dependencies in a sample application. If needed I can create a repo with a sample application that consumes the package.


  • I have considered if this change warrants release notes and have added them to the appropriate section in the CHANGELOG.md if necessary.

@@ -967,8 +967,7 @@
F1DC21B22199D58B007C189E /* ElementToAttributedString */,
F1E75630215B0C9E004BC254 /* StringAttributesToAttributes */,
);
name = Converters;
path = "New Group";
path = Converters;
Copy link
Author

Choose a reason for hiding this comment

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

I noticed that this group was not properly renamed on disk and renamed it so it shows up correctly in SPM.

Copy link
Author

Choose a reason for hiding this comment

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

This file is not part of the Xcode project, but SPM was picking it up and failing to compile.

Copy link
Author

Choose a reason for hiding this comment

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

This file is not part of the Xcode project, but SPM was picking it up and failing to compile.

@allenhumphreys
Copy link
Author

I just want to say thanks for the awesome library. I know I didn't ask about adding this support before opening the PR, it's ok if this doesn't align with your goal, though. I had trouble finding your project when looking to solve what this solves, and I'd love to get it listed in the swift package index if this goes through.

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.

Please support SPM~~~
1 participant