Skip to content

README: update 2 broken hyperlinks#5007

Merged
urvangjoshi merged 2 commits into
AOMediaCodec:mainfrom
urvangjoshi:readme_links
May 26, 2026
Merged

README: update 2 broken hyperlinks#5007
urvangjoshi merged 2 commits into
AOMediaCodec:mainfrom
urvangjoshi:readme_links

Conversation

@urvangjoshi
Copy link
Copy Markdown
Contributor

@urvangjoshi urvangjoshi commented May 22, 2026

Also remove the [TOC], that is specific to Gitlab.

@urvangjoshi urvangjoshi enabled auto-merge (squash) May 23, 2026 00:47
@FooIbar
Copy link
Copy Markdown

FooIbar commented May 24, 2026

Should the [TOC] in README be removed since it's GitLab-specific?

@urvangjoshi
Copy link
Copy Markdown
Contributor Author

Should the [TOC] in README be removed since it's GitLab-specific?

Thank you for your suggestion! Removed it in the updated PR.

Copy link
Copy Markdown

@andrey-norkin andrey-norkin left a comment

Choose a reason for hiding this comment

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

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

@urvangjoshi urvangjoshi merged commit c1a34e8 into AOMediaCodec:main May 26, 2026
109 checks passed
@urvangjoshi
Copy link
Copy Markdown
Contributor Author

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

Thanks for the review @andrey-norkin . Looks like PR was auto-merged after approval.
I'll prepare a separate PR to add link to CTC doc, if that's desired.

@urvangjoshi urvangjoshi deleted the readme_links branch May 26, 2026 20:30
@urvangjoshi
Copy link
Copy Markdown
Contributor Author

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

Thanks for the review @andrey-norkin . Looks like PR was auto-merged after approval. I'll prepare a separate PR to add link to CTC doc, if that's desired.

@andrey-norkin : here you go: #5012

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