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

Feature: Documentation refactor #204

Merged
merged 6 commits into from
Jul 16, 2024
Merged

Feature: Documentation refactor #204

merged 6 commits into from
Jul 16, 2024

Conversation

seangwright
Copy link
Member

@seangwright seangwright commented Jul 11, 2024

Motivation

  • Update docs to more closely match the repo-template
  • Add migration path diagram images
  • Add VS Code editor settings and recommended extensions
  • Disable dotnet format in CI pipeline
    • We can re-enable once all WIP branches have been merged and we have a single "format" PR
  • Update .editorconfig to match other integration repos

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

No functional changes have been made.

The README and other markdown can be previewed here.

@seangwright seangwright changed the title Feature: Documentation refator Feature: Documentation refactor Jul 11, 2024
@tkrch
Copy link
Collaborator

tkrch commented Jul 16, 2024

@seangwright i will perform solution reformat with dotnet format in PR #208 after updating editor config from https://github.com/Kentico/repo-template. Do You agree?

@seangwright
Copy link
Member Author

seangwright commented Jul 16, 2024

@tkrch yup! After this PR is merged into master, you can merge master in and pick up the .editorconfig changes.

@seangwright seangwright merged commit c06b19c into master Jul 16, 2024
2 checks passed
@seangwright seangwright deleted the feat/docs-refactor branch July 16, 2024 16:53
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