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.

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

chore(EMS-662): update README.md #180

Merged
merged 4 commits into from
Dec 9, 2022
Merged

chore(EMS-662): update README.md #180

merged 4 commits into from
Dec 9, 2022

Conversation

ttbarnes
Copy link
Contributor

@ttbarnes ttbarnes commented Dec 8, 2022

This PR adds more details about EXIP, the code structure/approach and some development principles.

Link to the markdown generation for easier reading: https://github.com/UK-Export-Finance/exip/blob/603970f99be82a24c6dff4fb120fa35dd940ae2e/README.md

@ttbarnes ttbarnes added the documentation Improvements or additions to documentation label Dec 8, 2022
@ttbarnes ttbarnes self-assigned this Dec 8, 2022
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:22 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:22 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:22 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:22 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 8, 2022 16:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes marked this pull request as ready for review December 8, 2022 16:36
abhi-markan
abhi-markan previously approved these changes Dec 8, 2022
Copy link
Collaborator

@abhi-markan abhi-markan left a comment

Choose a reason for hiding this comment

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

👍🏼 LGTM
Good effort @ttbarnes for an improved documentation, once I have finished all CI/CD and IaC tickets I will use the similar template and will update accordingly.

Well done.

@ttbarnes
Copy link
Contributor Author

ttbarnes commented Dec 8, 2022

Thanks @abhi-markan most appreciated - especially useful for new starters 😊 Sounds great re CI/CD/IaC.

Found a couple of typos with fresh eyes, will fix asap. Also ideally would like @Zainzzkk and @IgnatG eyes on this before merging 🙏

@ttbarnes ttbarnes requested a review from IgnatG December 8, 2022 18:53
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@ttbarnes ttbarnes temporarily deployed to dev December 9, 2022 09:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 9, 2022 09:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 9, 2022 09:33 — with GitHub Actions Inactive
@ttbarnes ttbarnes temporarily deployed to dev December 9, 2022 09:33 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Collaborator

@Zainzzkk Zainzzkk left a comment

Choose a reason for hiding this comment

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

LGTM

@ttbarnes ttbarnes merged commit c87a4c6 into main Dec 9, 2022
@ttbarnes ttbarnes deleted the chore/EMS-662 branch December 9, 2022 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants