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

changelog: spell fixes #198

Merged
merged 1 commit into from
Feb 18, 2024
Merged

changelog: spell fixes #198

merged 1 commit into from
Feb 18, 2024

Conversation

davlgd
Copy link
Collaborator

@davlgd davlgd commented Feb 18, 2024

Checklist

  • My PR is related to an opened issue : #

⚠️ If changes affect GitHub Action files

  • I have added tests to cover my changes : link

Pull request type

Please check the type of change your PR introduces:

  • Documentation content changes
  • Bugfix on the site
  • Build related changes
  • Other (please describe):

Description

Some spell fixes

@davlgd davlgd self-assigned this Feb 18, 2024
Copy link

Deployment has finished 👁️👄👁️ Your app is available here

@cnivolle cnivolle merged commit e0adc26 into main Feb 18, 2024
3 of 5 checks passed
Copy link

Your review app has been deleted 👋

juliamrch pushed a commit that referenced this pull request Feb 21, 2024
@davlgd davlgd deleted the davlgd-changelog-spell-fixes branch February 26, 2024 19:51
cnivolle pushed a commit that referenced this pull request Feb 27, 2024
* doc: fic Moodle guide according to vale linter recommendations

* fix passive voice

* doc: update link to WP-Bedrock github tutorial

* changelog: spell fixes (#198)

* changelog: Clever Tools 3.4.0 (#196)

* Simplify PR template

Former template was too detailed and not suited for quick changes like the changelog.

Furthermore, already pre-filled templates seem to reduce author's verbosity. Since there isn't any way of enforcing the filling of a template on PR's like on issues on GitHub, if any additional information is needed it should be requested by the reviewer.

* Update Hextra theme to v0.7.3

Fixe #183

* fix(jenkins): remove pricing information & components

* chore: relocate partials to get proper TOC (iss98)

* chore: relocate partials to get proper TOC (iss98)

* chore: reindex partials to get proper TOC

* chore: replace in codebase every {{< readfile file=file.md >}} used in Laravel tutorial by {{% content/file %}}

* doc: Rewrite Overview and remove 'create-app' partial (not specific enough for Laravel)

* doc-linting: rewrite partials and include new terms in Vale

* doc/laravel: rewrite Laravel guide

- Remove partials: partials are not specific enough for this guide and their style is too vague for specific frameworks.
- Relocate partials: rewriting implied to relocate partials to get proper TOCs in pages, as sepcified in iss98.
- Set guide in steps: use the steps shortcode to clarify the configuration and deployment process

Discussion: Rewriting partials or deprecating their usage for the most part.

* fix link to Bedrock repo

* Update create-application.md

add space before title to try to fix the content rendering

---------

Co-authored-by: David Legrand <1110600+davlgd@users.noreply.github.com>
Co-authored-by: Florian Sanders <florian.sanders@clever-cloud.com>
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.

2 participants