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

Adopt MIAF shared constraints #240

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Conversation

leo-barnes
Copy link
Collaborator

@leo-barnes leo-barnes commented Sep 12, 2024

This closes #169


Preview | Diff

Copy link
Collaborator

@y-guyon y-guyon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be in "the common constraints in the section [[#brands]]" to avoid the repetition

@leo-barnes
Copy link
Collaborator Author

Could be in "the common constraints in the section [[#brands]]" to avoid the repetition

See the comment inline. The reason MIAF did it this way is so that you can create a generic non-profiled MIAF file as well as MIAF files under specific profiles.

@y-guyon
Copy link
Collaborator

y-guyon commented Sep 12, 2024

See the comment inline. The reason MIAF did it this way is so that you can create a generic non-profiled MIAF file as well as MIAF files under specific profiles.

Oh okay, I did not connect the two things. It makes sense now.

@leo-barnes leo-barnes force-pushed the u/lbarnes/adopt_constraints branch from b16e9cc to ed4fdcb Compare September 25, 2024 07:03
@leo-barnes leo-barnes force-pushed the u/lbarnes/adopt_constraints branch from acc68f6 to 4215905 Compare September 25, 2024 07:06
@leo-barnes leo-barnes merged commit 1796dae into master Sep 25, 2024
@leo-barnes leo-barnes deleted the u/lbarnes/adopt_constraints branch September 25, 2024 07:07
github-actions bot added a commit that referenced this pull request Sep 25, 2024
SHA: 1796dae
Reason: push, by leo-barnes

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AVIF omits to select shared constraints from MIAF
2 participants