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

Add support and bump to mike 2.0.0 #174

Merged
merged 3 commits into from
Nov 6, 2023
Merged

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Nov 6, 2023

This PR upgrades mike to 2.0.0. This new version uses symlinks for aliases by default, but they are not supported by GitHub Pages, so we need to use the redirect alias type instead explicitly.

We use the new alias_type option in the mike plugin configuration in the mkdocs.yml file to set the alias type to redirect.

It also avoid failing the documentation publishing job if the mike version can't be determined (for example if the branch name is not recognized). It will just emit a warning and skip the publishing.

Fixes #173.

@llucax llucax requested a review from a team as a code owner November 6, 2023 09:04
@github-actions github-actions bot added the part:tests Affects the unit, integration and performance (benchmarks) tests label Nov 6, 2023
@llucax llucax self-assigned this Nov 6, 2023
@llucax llucax added part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users part:cookiecutter Affects the generation of projects using cookiecutter part:mkdocs Affects the configuration of mkdocs labels Nov 6, 2023
@llucax llucax enabled auto-merge November 6, 2023 09:06
@llucax llucax mentioned this pull request Nov 6, 2023
@github-actions github-actions bot removed part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:cookiecutter Affects the generation of projects using cookiecutter part:mkdocs Affects the configuration of mkdocs labels Nov 6, 2023
@llucax llucax added part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:bug Something isn't working part:cookiecutter Affects the generation of projects using cookiecutter part:mkdocs Affects the configuration of mkdocs type:regression Something isn't working, but it was working before labels Nov 6, 2023
Signed-off-by: Leandro Lucarella <[email protected]>
This commit upgrades `mike` to 2.0.0. This new version uses symlinks
for aliases by default, but they are not supported by GitHub Pages, so
we need to use the `redirect` alias type instead explicitly.

We use the new `alias_type` option in the `mike` plugin configuration
in the `mkdocs.yml` file to set the alias type to `redirect`.

Signed-off-by: Leandro Lucarella <[email protected]>
@github-actions github-actions bot removed part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:cookiecutter Affects the generation of projects using cookiecutter part:mkdocs Affects the configuration of mkdocs labels Nov 6, 2023
If the `mike` version can't be determined, just emit a warning and exit
succesfully. This is useful for the CI, where the `mike` version can't
be determined because the branch name doesn't follow the guidelines (for
example in PRs).

The CI will now emit a warning and skip the publishing of the
documentation website.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax added this pull request to the merge queue Nov 6, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit 7e74141 Nov 6, 2023
14 checks passed
@llucax llucax deleted the mike2 branch November 6, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:tests Affects the unit, integration and performance (benchmarks) tests type:bug Something isn't working type:regression Something isn't working, but it was working before type:tech-debt Improves the project without visible changes for users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for mike 2
2 participants