Skip to content

Adopt Docs Starter Pack#429

Merged
slyon merged 3 commits intocanonical:mainfrom
rkratky:FR-6115_starter-pack
Jan 8, 2024
Merged

Adopt Docs Starter Pack#429
slyon merged 3 commits intocanonical:mainfrom
rkratky:FR-6115_starter-pack

Conversation

@rkratky
Copy link
Contributor

@rkratky rkratky commented Dec 14, 2023 •
edited
Loading

Description

  • Exclude changes in 'doc/' dir from code CI checks.
  • Adopt Docs Starter Pack:
    • Add CI linkcheck
    • Substitute spellcheck & incl. lang. check w/ solution from Starter Pack
    • Fix ton of spelling mistakes
    • Update & fix markup for consistency
    • Use redirects instead of symlinks
  • Fix Doxygen config. & remove Doxyfile.

Checklist

  • Runs make check successfully.

  • Retains 100% code coverage (make check-coverage).

  • N/A: New/changed keys in YAML format are documented.

  • N/A: (Optional) Adds example YAML for new feature.

  • (Optional) Closes an open bug in Launchpad. -- Fixes FR-6115, FR-6157

@rkratky rkratky added enhancement New feature or request documentation Documentation improvements. labels Dec 14, 2023
@rkratky rkratky requested review from daniloegea and slyon December 14, 2023 18:05
@rkratky rkratky force-pushed the FR-6115_starter-pack branch 2 times, most recently from 7496a64 to 9dd4239 Compare December 14, 2023 18:26
@rkratky rkratky changed the title Adopt Docs Starter Pack [WIP] Adopt Docs Starter Pack Dec 14, 2023
@rkratky rkratky force-pushed the FR-6115_starter-pack branch 7 times, most recently from 28c1a52 to 33556d7 Compare December 14, 2023 23:10
@rkratky rkratky changed the title [WIP] Adopt Docs Starter Pack Adopt Docs Starter Pack Dec 14, 2023
@rkratky rkratky mentioned this pull request Dec 15, 2023
Merged
5 tasks
daniloegea
daniloegea reviewed Jan 2, 2024
View reviewed changes
daniloegea
daniloegea reviewed Jan 2, 2024
View reviewed changes
daniloegea
daniloegea reviewed Jan 2, 2024
View reviewed changes
daniloegea
daniloegea reviewed Jan 2, 2024
View reviewed changes
daniloegea
daniloegea reviewed Jan 2, 2024
View reviewed changes
Copy link
Contributor

@daniloegea daniloegea left a comment

Choose a reason for hiding this comment

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

Thanks, Robert. It looks good. I left some minor comments. Let's see what Lukas says.

@rkratky rkratky force-pushed the FR-6115_starter-pack branch 4 times, most recently from 3efa157 to 720021e Compare January 3, 2024 11:09
slyon
slyon reviewed Jan 3, 2024
View reviewed changes
Copy link
Contributor

@slyon slyon left a comment

Choose a reason for hiding this comment

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

Thank you Robert! This is looking mostly good to me. I left a few comments & questions inline.

@rkratky rkratky force-pushed the FR-6115_starter-pack branch from 720021e to 9090a88 Compare January 5, 2024 14:25
@rkratky
Copy link
Contributor Author

rkratky commented Jan 5, 2024

@slyon I addressed all the comments, and I believe this is good top merge. Thanks for the review.

@slyon
Copy link
Contributor

slyon commented Jan 8, 2024

I only have one question/concern before merging, about the woke implementation.
AFAICT the new woke check only checks the docs inside doc/, whereas our old implementation (#303) actually checked all of the source code (i.e. public headers/API, ...). Maybe we should just keep that original check around, additionally? I.e. not removing this: https://github.com/canonical/netplan/pull/429/files#diff-651a780d1a032042f88eece53425bb26377cd39c299fa9dafbd643b126d6981f

WDYT? Would this work?

@rkratky rkratky force-pushed the FR-6115_starter-pack branch from 9090a88 to e01451c Compare January 8, 2024 14:02
rkratky added 2 commits January 8, 2024 16:19
@rkratky
Adopt Docs Starter Pack:
d1efaff 
- Add CI linkcheck
- Substitute spellcheck & incl. lang. check w/ solution
      from Starter Pack
- Fix ton of spelling mistakes
- Update & fix markup for consistency
- Use redirects instead of symlinks
@rkratky
Fix Doxygen config. & remove Doxyfile.
e208814
@rkratky rkratky force-pushed the FR-6115_starter-pack branch from e01451c to e208814 Compare January 8, 2024 15:20
slyon
slyon approved these changes Jan 8, 2024
View reviewed changes
Copy link
Contributor

@slyon slyon left a comment

Choose a reason for hiding this comment

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

Thanks Robert, for your contribution! With all the fixes landed, this LGTM!

@slyon slyon merged commit d3a408c into canonical:main Jan 8, 2024
@rkratky rkratky deleted the FR-6115_starter-pack branch January 8, 2024 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@daniloegea daniloegea daniloegea left review comments

@slyon slyon slyon approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Documentation improvements. enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rkratky @slyon @daniloegea