Skip to content

Fix Typo: Remove duplicated "not" #2366

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

takahirox
Copy link
Contributor

No description provided.

@javagl
Copy link
Contributor

javagl commented Apr 3, 2024

@lexaknyazev When things affect the core spec, then I'd hesitate to just review/merge anything without your approval (a similar case as in #2351 ). If you say that things like these (uncontroversial typos) are OK to be handled by anyone, then these PRs could be worked off more quickly.

@emackey emackey added this to the 2.0.x milestone Aug 2, 2024
@javagl
Copy link
Contributor

javagl commented Mar 15, 2025

@lexaknyazev If you agree that things like

can be merged at any point in time and by others than you, then this could help work off the PR queue.

But maybe there is a plan/schedule that should be followed.

For example: If the plan is that such PRs are consistently tagged PRs specification and added to https://github.com/KhronosGroup/glTF/milestone/4 , so that they can be merged "in bulk" (and maybe to create a change log from that), then it would be good to know this process.

(If there is such a process, then I'd add a comment to such PRs, summarizing that: "This is approved and will be merged a part of [...]", just to let those know who opened them)

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