Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Formatting, Typos #149

Merged
merged 4 commits into from
Apr 30, 2024
Merged

Formatting, Typos #149

merged 4 commits into from
Apr 30, 2024

Conversation

OG-NI
Copy link
Contributor

@OG-NI OG-NI commented Apr 30, 2024

  • fixed formatting, typos
  • added prettier config for docs
  • added npm scripts and github action to check formatting of docs for each PR

@OG-NI OG-NI requested a review from benedictweis April 30, 2024 14:53
@OG-NI
Copy link
Contributor Author

OG-NI commented Apr 30, 2024

@benedictweis please check if the github action is correct

Copy link
Contributor

@benedictweis benedictweis left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes, the PR looks great. I have added some things regarding the action, please see my comments.

.github/workflows/docs.yml Outdated Show resolved Hide resolved
.github/workflows/docs.yml Show resolved Hide resolved
.github/workflows/docs.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@benedictweis benedictweis left a comment

Choose a reason for hiding this comment

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

The action looks good, lets merge and find out if it works in the next PR 😉.

@OG-NI OG-NI merged commit 470d35c into main Apr 30, 2024
3 checks passed
@OG-NI OG-NI deleted the fix/formatting-typos branch April 30, 2024 15:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants