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

Upgrade spdlog to 1.10.0 #312

Merged
merged 1 commit into from
Dec 2, 2022
Merged

Conversation

kkraus14
Copy link
Contributor

@kkraus14 kkraus14 commented Dec 1, 2022

Description

Upgrades the default spdlog version to 1.10.0 to match the conda-forge pinning.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@rapids-bot
Copy link

rapids-bot bot commented Dec 1, 2022

Pull requests from external contributors require approval from a rapidsai organization member with write or admin permissions before CI can begin.

@kkraus14 kkraus14 mentioned this pull request Dec 1, 2022
3 tasks
@robertmaynard robertmaynard added feature request New feature or request non-breaking Introduces a non-breaking change 3 - Ready for Review Ready for review by team Dependencies This PR updates a dependency in this repo labels Dec 1, 2022
@ajschmidt8
Copy link
Member

/ok to test

@robertmaynard
Copy link
Contributor

My plan is to merge this after rapidsai/rmm#1173 has been merged. If rmm is 'green' with 1.10 I see no problem updating spdlog for everyone else

rapids-bot bot pushed a commit to rapidsai/rmm that referenced this pull request Dec 2, 2022
Updates spdlog dependency to 1.10 to match the conda-forge pinning. Depends on rapidsai/rapids-cmake#312

Also updates the flake8 pre-commit hook location since it's migrated from gitlab to github.

Authors:
  - Keith Kraus (https://github.com/kkraus14)

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Bradley Dice (https://github.com/bdice)
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: #1173
@bdice
Copy link
Contributor

bdice commented Dec 2, 2022

RMM CI was green and I merged rapidsai/rmm#1173. I will merge this too.

@bdice
Copy link
Contributor

bdice commented Dec 2, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 4b079dc into rapidsai:branch-23.02 Dec 2, 2022
bdice added a commit to bdice/rapids-cmake that referenced this pull request Dec 5, 2022
rapids-bot bot pushed a commit that referenced this pull request Dec 6, 2022
This reverts commit 4b079dc.

See rapidsai/rmm#1176 for details.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #323
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team Dependencies This PR updates a dependency in this repo feature request New feature or request non-breaking Introduces a non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants