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.

Sign up for GitHub

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

Jump to bottom

Add unicode doc to doxygen Tech Notes #5136

Merged
merged 5 commits into from
Nov 22, 2024
Merged

Add unicode doc to doxygen Tech Notes #5136

merged 5 commits into from
Nov 22, 2024

Conversation

byrnHDF
Copy link
Contributor

@byrnHDF byrnHDF commented Nov 20, 2024

No description provided.

@byrnHDF byrnHDF added Priority - 1. High 🔼 These are important issues that should be resolved in the next release Component - Documentation Doxygen, markdown, etc. Type - Improvement Improvements that don't add a new feature or functionality labels Nov 20, 2024
@byrnHDF byrnHDF self-assigned this Nov 20, 2024
hyoklee
hyoklee requested changes Nov 21, 2024
View reviewed changes
doxygen/dox/TechnicalNotes.dox Outdated Show resolved Hide resolved
hyoklee
hyoklee reviewed Nov 21, 2024
View reviewed changes
Copy link
Member

@hyoklee hyoklee left a comment

Choose a reason for hiding this comment

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

Please minimize the use of 8-bit characters in code documentation.

@byrnHDF
Copy link
Contributor Author

byrnHDF commented Nov 21, 2024

Please minimize the use of 8-bit characters in code documentation.

These are all conversions from html/md files and it can be difficult to catch everything - appreciate the checks everyone does!

@lrknox lrknox merged commit e16f47b into HDFGroup:develop Nov 22, 2024
67 checks passed
@byrnHDF byrnHDF deleted the develop-doxy-uni branch November 25, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lrknox lrknox lrknox approved these changes

@hyoklee hyoklee hyoklee approved these changes

@derobins derobins Awaiting requested review from derobins derobins is a code owner

@fortnern fortnern Awaiting requested review from fortnern fortnern is a code owner

@jhendersonHDF jhendersonHDF Awaiting requested review from jhendersonHDF jhendersonHDF is a code owner

@qkoziol qkoziol Awaiting requested review from qkoziol qkoziol is a code owner

@vchoi-hdfgroup vchoi-hdfgroup Awaiting requested review from vchoi-hdfgroup vchoi-hdfgroup is a code owner

@bmribler bmribler Awaiting requested review from bmribler bmribler is a code owner

@glennsong09 glennsong09 Awaiting requested review from glennsong09 glennsong09 is a code owner

@mattjala mattjala Awaiting requested review from mattjala mattjala is a code owner

@brtnfld brtnfld Awaiting requested review from brtnfld brtnfld is a code owner

Assignees

@byrnHDF byrnHDF

Labels
Component - Documentation Doxygen, markdown, etc. Priority - 1. High 🔼 These are important issues that should be resolved in the next release Type - Improvement Improvements that don't add a new feature or functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@byrnHDF @hyoklee @lrknox