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

Gstro/update dependencies #19

Merged
merged 2 commits into from
May 10, 2018
Merged

Gstro/update dependencies #19

merged 2 commits into from
May 10, 2018

Conversation

gstro
Copy link
Contributor

@gstro gstro commented May 10, 2018

Update Cartfile to point to Heimdallr commit

  • Heimdallr updated their master branch (currently at specified commit)
    which drops their dependency on outdated libraries. But they have not
    made a new release for this change -- see issue here:
    New release based on master? trivago/Heimdallr.swift#118
  • This prevents conflicts for Carthage users where they are using
    updated versions of those libraries.
  • The podspec file cannot be updated in the same way, but it may not be
    an issue for CocoaPods users.
  • Once Heimdallr has bumped their release, we can return to proper
    dependency versioning.
  • Also adds baselines for some benchmark tests

Greg Stromire added 2 commits May 10, 2018 10:47
- Heimdallr updated their master branch (currently at specified commit)
  which drops their dependency on outdated libraries. But they have not
  made a new release for this change -- see issue here:
  trivago/Heimdallr.swift#118
- This prevents conflicts for Carthage users where they are using
  updated versions of those libraries.
- The podspec file cannot be updated in the same way, but it may not be
  an issue.
- Once Heimdallr has bumped their release, we can return to proper
  dependency versioning.
@gstro gstro merged commit 9c4c28b into master May 10, 2018
@gstro gstro deleted the gstro/update-dependencies branch May 10, 2018 21:20
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