Skip to content

Update README to reflect current v2 repo state#427

Open
Copilot wants to merge 3 commits intomasterfrom
copilot/update-readme-to-reflect-current-repo-state
Open

Update README to reflect current v2 repo state#427
Copilot wants to merge 3 commits intomasterfrom
copilot/update-readme-to-reflect-current-repo-state

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

Motivation

The README was stale: wrong version, mentioned Fortran utilities that no longer exist, outdated cmake example (included CMAKE_Fortran_COMPILER), and missing coverage of sub-packages and cmake utility scripts.

Changes

  • General description — Corrected to "C/C++ library"; removed Fortran references.
  • Sub-packages table — Lists all 7 packages (Core, KokkosUtils, Pack, Algorithm, Expression, Logging, Parser) with CMake target names and 1-liner descriptions.
  • Build system — Updated cmake example (uses EKAT_ENABLE_ALL_PACKAGES, no Fortran compiler); replaced bullet list of options with a structured table.
  • CMake utility scripts — New section documenting EkatCreateUnitTest.cmake, EkatSetCompilerFlags.cmake, EkatUtils.cmake, EkatMpiUtils.cmake, and the tpls/ fetch/build scripts.

E3SM Stakeholder Feedback

N/A

Testing

Documentation-only change; no code paths affected.

Copilot AI linked an issue Apr 8, 2026 that may be closed by this pull request
Copilot AI and others added 2 commits April 8, 2026 19:56
Agent-Logs-Url: https://github.com/E3SM-Project/EKAT/sessions/aff84f20-e509-4dbc-ad39-f3c3122b7faf

Co-authored-by: bartgol <3226046+bartgol@users.noreply.github.com>
Agent-Logs-Url: https://github.com/E3SM-Project/EKAT/sessions/aff84f20-e509-4dbc-ad39-f3c3122b7faf

Co-authored-by: bartgol <3226046+bartgol@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README to reflect current repo state Update README to reflect current v2 repo state Apr 8, 2026
Copilot AI requested a review from bartgol April 8, 2026 19:58
@bartgol bartgol marked this pull request as ready for review April 8, 2026 20:25
Copy link
Copy Markdown
Contributor

@bartgol bartgol left a comment

Choose a reason for hiding this comment

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

@jgfouca @jeff-cohere @tcclevenger feel free to add/modify the README. To me, it looks like a reasonable version.

Copy link
Copy Markdown
Contributor

@jeff-cohere jeff-cohere left a comment

Choose a reason for hiding this comment

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

lgtm

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.

Update README to reflect current repo state

3 participants