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

Feb key vault release #4319

Merged
merged 13 commits into from
Feb 7, 2023
Merged

Feb key vault release #4319

merged 13 commits into from
Feb 7, 2023

Conversation

gearama
Copy link
Member

@gearama gearama commented Feb 1, 2023

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

  • C++ Guidelines
  • Doxygen docs
  • Unit tests
  • No unwanted commits/changes
  • Descriptive title/description
    • PR is single purpose
    • Related issue listed
  • Comments in source
  • No typos
  • Update changelog
  • Not work-in-progress
  • External references or docs updated
  • Self review of PR done
  • Any breaking changes?

@antkmsft
Copy link
Member

antkmsft commented Feb 2, 2023

I think we rather should update the core minimum version from "1.5.0" to "1.8.0" in azure-security-keyvault-administration/vcpkg/Config.cmake.in and in azure-security-keyvault-administration/vcpkg/vcpkg.json, unless you know that it will compile and work with "1.5.0". (sorry, initially I also suggested to update CMakeLists - that one we should not do)

@gearama
Copy link
Member Author

gearama commented Feb 2, 2023

but 1.8.0 has not been released, we are also releasing 1.8.0-beta4 , the last GA one is 1.7.2

@antkmsft
Copy link
Member

antkmsft commented Feb 2, 2023

@gearama, we are releasing Core 1.8.0 tomorrow. But sorry for the rather bad suggestion about updating the CMakeLists.txt. But the other two files from the /vcpkg directory is a good update. I think you currently have the PR in ideal state.

@gearama gearama merged commit 39ec586 into Azure:main Feb 7, 2023
antkmsft pushed a commit that referenced this pull request Apr 5, 2023
* kv release

* ds

* Update sdk/keyvault/azure-security-keyvault-administration/CHANGELOG.md

Co-authored-by: Ahson Khan <[email protected]>

* core to latest GA

* 1.8.0

* update folder list

* put to stable versions

* put back to last GA

* put back to original

* try core 1.8.0

* Revert "try core 1.8.0"

This reverts commit 1e0b612.

---------

Co-authored-by: Ahson Khan <[email protected]>
@gearama gearama deleted the FebKVRelease branch January 16, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants