Skip to content

Add CI Configuration part for enable creation date in git-revision-date-localized plugin#6015

Closed
mendax1234 wants to merge 0 commit intosquidfunk:masterfrom
mendax1234:master
Closed

Add CI Configuration part for enable creation date in git-revision-date-localized plugin#6015
mendax1234 wants to merge 0 commit intosquidfunk:masterfrom
mendax1234:master

Conversation

@mendax1234
Copy link
Copy Markdown
Contributor

I am not sure whether it is suitable to use note here, but if not, let me know and it can be changed at any time before merging.
Also, l think a simple example here would be better. However, l think doing another complete section about adding CI workflow to your documentation site would be much better. Since l am not an expert at the CI workflow, l think maybe l can just link to the CI workflow configuration part if it is done.

@alexvoss
Copy link
Copy Markdown
Collaborator

Thanks for this. Not everything is using a CI system, though, so perhaps add a sentence that says that this is needed if people are deploying through a CI system? (M4MkDocs is used by a very wide range of users.)

Comment thread docs/setup/adding-a-git-repository.md Outdated

[With Github Actions](../publishing-your-site.md#with-github-actions):

=== "Material for MkDocs"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please don't copy our entire CI setup guide, as we'd now have two places where we have to maintain it. Please just clarify with a sentence that when building in a CI environment, you might need to adjust the setting when fetching the code. That's exactly what @alexvoss mentioned. Thanks.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok, thanks! I will change it.

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