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

Clarify to localizers git tree SHA vs git commit SHA. #1239

Merged
merged 5 commits into from Nov 1, 2023

Conversation

BillyONeal
Copy link
Member

Another extraction from #1210 .

include/vcpkg/base/message-data.inc.h Outdated Show resolved Hide resolved
include/vcpkg/base/message-data.inc.h Outdated Show resolved Hide resolved
include/vcpkg/base/message-args.inc.h Outdated Show resolved Hide resolved
include/vcpkg/base/message-data.inc.h Outdated Show resolved Hide resolved
# Conflicts:
#	include/vcpkg/base/message-data.inc.h
#	locales/messages.json
#	src/vcpkg/vcpkgpaths.cpp
@BillyONeal BillyONeal added the depends:different-pr This PR depends on a different PR which has been filed label Oct 30, 2023
@BillyONeal
Copy link
Member Author

Depends on #1260

include/vcpkg/base/messages.h Outdated Show resolved Hide resolved
BillyONeal added a commit that referenced this pull request Oct 31, 2023
* Delocalize error: warning: and note:.

Response to code review comment #1239 (comment)
@BillyONeal BillyONeal removed the depends:different-pr This PR depends on a different PR which has been filed label Oct 31, 2023
# Conflicts:
#	include/vcpkg/base/checks.h
#	include/vcpkg/base/messages.h
#	src/vcpkg/base/checks.cpp
#	src/vcpkg/base/hash.cpp
#	src/vcpkg/base/message_sinks.cpp
#	src/vcpkg/base/messages.cpp
#	src/vcpkg/base/parse.cpp
#	src/vcpkg/commands.z-generate-message-map.cpp
#	src/vcpkg/registries.cpp
#	src/vcpkg/vcpkgpaths.cpp
@BillyONeal BillyONeal merged commit da950b3 into microsoft:main Nov 1, 2023
5 checks passed
@BillyONeal BillyONeal deleted the git-tree-messages branch November 1, 2023 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants