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

daw_json_link: add version 3.28.3 #25316

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

toge
Copy link
Contributor

@toge toge commented Sep 19, 2024

Summary

Changes to recipe: daw_json_link/*

Motivation

There are several improvements since 3.24.1.

Details

beached/daw_json_link@v3.24.1...v3.28.3

Because daw_json_link and daw_utf_range requires daw_utf_header, we have to update daw_utf_range and daw_utf_header at same time.


@conan-center-bot

This comment has been minimized.

@toge
Copy link
Contributor Author

toge commented Sep 19, 2024

@AbrilRBS
Could you help me?

The extra attributes for dependencies have been introduced by #24499.
It worked fine on CI at that moment.

Are there any white list to check attributes of conandata.yml?

@AbrilRBS
Copy link
Member

AbrilRBS commented Oct 5, 2024

@toge sorry for the dealy, I didn't see your notification. This is quite weird because that same pattern is working elsewhere 🤔

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Warning

Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement.

conandata.yml error in build 3:

Invalid attributes found: daw_headers_mapping

Conan v2 pipeline ❌

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping @conan-io/barbarians on the PR and we will help you.

conandata.yml error in build 3:

Invalid attributes found: daw_headers_mapping

@AbrilRBS
Copy link
Member

Will need to double check with the team as to what might be going on here! Locally this works!

@jcar87 jcar87 self-assigned this Nov 4, 2024
@AbrilRBS
Copy link
Member

AbrilRBS commented Nov 4, 2024

Thanks @toge, this now works with the new pipeline, so merging as-is :)

@AbrilRBS AbrilRBS merged commit 391fc55 into conan-io:master Nov 4, 2024
8 of 10 checks passed
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.

5 participants