Skip to content

Changes made to application process#281

Merged
nickaj merged 6 commits intoEPCCed:mainfrom
davyepcc:main
Nov 20, 2025
Merged

Changes made to application process#281
nickaj merged 6 commits intoEPCCed:mainfrom
davyepcc:main

Conversation

@davyepcc
Copy link
Copy Markdown
Contributor

@davyepcc davyepcc commented Nov 11, 2025

EIDF Documentation Pull Request

Description

Update to application process

Fixes # (issue)

Type of change

Please delete options that are not relevant.

What has to be reviewed

Checklist

  • [x ] Documentation follows the project style guidelines
  • [ x] Ensure Contact details contain Service Emails and Numbers
  • [ x] Self-review of documentation using mkdocs on local system
  • [ x] Spellcheck has been performed
  • Pre-commit has been run and passed YES

@davyepcc
Copy link
Copy Markdown
Contributor Author

As noted in emails, pre-commit fix does not work as code barfed.

Copy link
Copy Markdown
Contributor Author

@davyepcc davyepcc left a comment

Choose a reason for hiding this comment

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

I've reviewed my changes.

Made changes to style issues
fixing more markup errrors
...more fixes
@davyepcc
Copy link
Copy Markdown
Contributor Author

AG and I fixed pre-commit.

All changes/mistakes corrected.

Thanks!

Copy link
Copy Markdown
Collaborator

@agngrant agngrant left a comment

Choose a reason for hiding this comment

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

Lines 31, 80, 82, 90, 92, 94, 128, 130, 134, 138 include mistypings. Can these be corrected?

fixed typos. oops.
Copy link
Copy Markdown
Contributor Author

@davyepcc davyepcc left a comment

Choose a reason for hiding this comment

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

typos fixed

@nickaj nickaj merged commit 994ddde into EPCCed:main Nov 20, 2025
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