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

Revert "Revert "json_parser: add idf_version rules for jsmn"" (IEC-29) #206

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

shahpiyushv
Copy link
Contributor

This reverts commit 7b08cd0.

The idf component manager now supports uploading components with rules.

Checklist

  • Component contains License
  • Component contains README.md
  • Component contains idf_component.yml file with url field defined
  • Component was added to upload job
  • Component was added to build job
  • Optional: Component contains unit tests
  • CI passing

Change description

This just reverts an earlier change which was added due to a bug in idf component manager, which did not allow external components to have "rules" in dependencies. Now that the bug is fixed, we can add back the rules.

@CLAassistant
Copy link

CLAassistant commented Jul 12, 2023

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot changed the title Revert "Revert "json_parser: add idf_version rules for jsmn"" Revert "Revert "json_parser: add idf_version rules for jsmn"" (IEC-29) Jul 12, 2023
@shahpiyushv
Copy link
Contributor Author

What needs to be done to get this merged?

This reverts commit 7b08cd0.

The idf component manager now supports uploading components with rules.
@mahavirj mahavirj merged commit 5558032 into espressif:master Jul 17, 2023
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants