Skip to content

feat: add developer instructions - #183

Open
Oisin-M wants to merge 2 commits into
developfrom
feat/dev_instructions
Open

feat: add developer instructions#183
Oisin-M wants to merge 2 commits into
developfrom
feat/dev_instructions

Conversation

@Oisin-M

@Oisin-M Oisin-M commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

Time has come to add some centralised developer instructions to the main earthkit packages that each of the subpackages can link to. Idea is to have as much as possible in the central instructions and as little as possible subpackage-specific instructions so it's more seamless to contribute across the stack and additionally also hopefully even more seamless experience when using the different components.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@EddyCMWF EddyCMWF left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved from a technical implementation and general structure point of view.

@Oisin-M
Oisin-M requested a review from iainrussell July 31, 2026 13:58
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.

3 participants