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 a readme file to workflows #5334

Merged
merged 7 commits into from
Feb 26, 2025
Merged

Add a readme file to workflows #5334

merged 7 commits into from
Feb 26, 2025

Conversation

byrnHDF
Copy link
Contributor

@byrnHDF byrnHDF commented Feb 25, 2025

No description provided.

@byrnHDF byrnHDF added Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. Component - Documentation Doxygen, markdown, etc. Type - Improvement Improvements that don't add a new feature or functionality Component - Misc Anything else (CODEOWNERS, etc.) labels Feb 25, 2025
@byrnHDF byrnHDF self-assigned this Feb 25, 2025
hyoklee
hyoklee reviewed Feb 25, 2025
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 remove contents related to Autotools.

@byrnHDF
Copy link
Contributor Author

byrnHDF commented Feb 25, 2025

Please remove contents related to Autotools.

Not until autotools is removed

brtnfld
brtnfld reviewed Feb 25, 2025
View reviewed changes
Copy link
Collaborator

@brtnfld brtnfld left a comment

Choose a reason for hiding this comment

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

I was also hoping for a short description of the YML files so that could be easily referenced since some of the files lack that information.

@byrnHDF
Copy link
Contributor Author

byrnHDF commented Feb 25, 2025

I was also hoping for a short description of the YML files so that could be easily referenced since some of the files lack that information.

Yeah, I should have done that! Will update

@brtnfld brtnfld self-requested a review February 25, 2025 23:53
@lrknox lrknox merged commit 9ae4164 into HDFGroup:develop Feb 26, 2025
3 checks passed
@byrnHDF byrnHDF deleted the develop-wfl-readme branch February 26, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyoklee hyoklee hyoklee left review comments

@brtnfld brtnfld brtnfld approved these changes

@lrknox lrknox lrknox 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

Assignees

@byrnHDF byrnHDF

Labels
Component - Documentation Doxygen, markdown, etc. Component - Misc Anything else (CODEOWNERS, etc.) Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. 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.
4 participants
@byrnHDF @hyoklee @brtnfld @lrknox