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

Close #63 Allow --config-file option in mkdocs #64

Merged
merged 4 commits into from
May 15, 2023

Conversation

fgrbr
Copy link

@fgrbr fgrbr commented May 9, 2023

This PR allows the path to the mkdocs config-file to be relative and not only absolute.

@smarie
Copy link
Owner

smarie commented May 10, 2023

Great @fgrbr , thanks ! Could you please also add a line in the changelog ? You need to pull from main first since I just pushed the new changelog section for "in progress". Concerning style, refer to the previous entry for the previous merged PR (by GenevieveBuckley)

Optional: if you see an easy way to test this feature against non-regression, do not hesitate to add a test to this PR ! This would probably imply creating a mock object and injecting a fake config with the various path flavours inside.

@fgrbr
Copy link
Author

fgrbr commented May 10, 2023

Thanks for the quick response ! I've added a line to changelog and fixed a typo.

Unfortunately, my python skills are not good enough to add a useful test for mocking the different paths. Sorry.

@smarie smarie merged commit 804b19e into smarie:main May 15, 2023
13 checks passed
@smarie
Copy link
Owner

smarie commented May 15, 2023

Thanks @fgrbr !

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.

None yet

2 participants